AldarisPale

Results 39 comments of AldarisPale

Hi again. After getting deeper into python and learning that `__del__` is a bad idea [anyway](https://bugs.python.org/msg80855) I have refactored the code: ```python #!/usr/bin/python3 # -*- coding: utf8 -*- from contracts...

@g-bougard there's chromeos flex, which is installable on macbooks and pc laptops. https://support.google.com/chromeosflex/answer/11552529 Os installer direct link: https://dl.google.com/chromeos-flex/images/latest.bin.zip Needs to be unzipped and then dd'd to USB stick

Looks like getting rid of the specific error message is relatively simple: change https://github.com/DonutsNL/phpsaml/blob/0bb7abba655a79d21ee58cb329a3ff9d0067f78c/front/acs.php#L4 from `$_POST = '';` to `$_POST = [];` _but_ it looks like this is actually the...

Thanks, @DonutsNL. Working now. > I got error that samlResponse was not valid and browser gave "CORS Missing Allow Origin" errors, so something else must be broken too. Update about...

@DonutsNL can confirm that on GLPI 10.0.7 + latest master from https://github.com/DonutsNL/phpsaml results in empty page for /plugins/phpsaml/front/config.php Login does seem to work though even with modified https://github.com/derricksmith/phpsaml/issues/120#issuecomment-1525542541 applied on...

The error message is: `glpiphplog.CRITICAL: *** Uncaught Exception TypeError: Argument 1 passed to PluginPhpsamlConfig::requested_authn_context() must be of the type string, null given, called in /var/www/plugins/phpsaml/inc/config.class.php on line 162 in /var/www/plugins/phpsaml/inc/config.class.php...

Thanks, @DonutsNL . When I change row 889 at https://github.com/DonutsNL/phpsaml/blob/56a15f3aa923588d27f70c4cd6d00729f0a7b533/inc/config.class.php#L817 from `protected function saml_idp_single_logout_service(string $cValue) : void` to `protected function saml_idp_single_logout_service($cValue) : void` then /plugins/phpsaml/front/config.php does open, but with errors:...

Thanks, @DonutsNL , for looking into it. Also noticed a typo at https://github.com/DonutsNL/phpsaml/blob/56a15f3aa923588d27f70c4cd6d00729f0a7b533/install/update.class.php#L301 Instead of `Toolbox::logInFile("phpsaml", "INFO -- PHPSAML upgraded to 1.2.1" . "\n", true);` it should probably be `Toolbox::logInFile("phpsaml",...

@DonutsNL one more addition regarding glpi 10 - this time about fusioninventory -> glpi inventory / glpi agent migration. Here's a patch for a couple-days-old phpsaml version, does not directly...

@DonutsNL separate issue created: https://github.com/derricksmith/phpsaml/issues/134