phpsaml icon indicating copy to clipboard operation
phpsaml copied to clipboard

Cannot modify header information headers already sent by ...

Open jbtele29 opened this issue 2 years ago • 3 comments

Hello,

I have this error in php log on my glpi

[2023-09-28 17:16:01] glpiphplog. WARNING: *** PHP Warning (2): Cannot modify header information - headers already sent by (output started at /var/www/html/glpi/plugins/phpsaml/hook.php:113) in /var/www/html/glpi/front/inventory.php at line 110 Backtrace : front/inventory.php:110 header() ...tplace/glpiinventory/front/communication.php:72 include_once() marketplace/glpiinventory/index.php:45 include_once() public/index.php:82 require()

I use glpi-agent 1.5 and glpi 10.0.9

Thanks

jbtele29 avatar Sep 28 '23 15:09 jbtele29

Please use the latest version in my repository: https://github.com/DonutsNL/phpsaml. It has various excludes for the latest inventory agents not present in this version of the plugin. Rgrds,

DonutsNL avatar Dec 24 '23 21:12 DonutsNL

Hello, thanks for update but i still have same problem after upgrade with your latest version [2023-12-26 11:04:08] glpiphplog.WARNING: *** PHP Warning (2): Cannot modify header information - headers already sent by (output started at /var/www/html/glpi/plugins/phpsaml/hook.php:115) in /var/www/htm> Backtrace : front/inventory.php:112 header() ...tplace/glpiinventory/front/communication.php:72 include_once() marketplace/glpiinventory/index.php:45 include_once() public/index.php:82 require()

jbtele29 avatar Dec 26 '23 10:12 jbtele29

Sorry this is my bad.

I forgot to comment out some debugging lines in the partially implemented rule engine code. I commented out the problematic code. This should solve the problem. Be aware that the rules for assigning user properties is not yet implemented and will not yet work. Best not to create phpsaml rules at this moment.

DonutsNL avatar Dec 26 '23 11:12 DonutsNL