derricksmith / phpsaml

GLPI Plugin - SAML integration using the Onelogin SAML Library
MIT License
32 stars 24 forks source link

Unable to upload documents in GLPI when SAML is enable #123

Closed ilicilho closed 1 year ago

ilicilho commented 1 year ago

We discovery a interesting think. When SAML Plugin is enable, the feature Document Manager from GLPI works with limitations. Example When upload a document, the page /front/document.form.php keeps in blank screen. In apache logs I see:

PHP Warning: Declaration of PluginPhpsamlConfig::showForm() should be compatible with CommonDBTM::showForm($ID, array $options = Array) in /var/www/glpi/plugins/phpsaml/inc/config.class.php on line 53, referer: https://mydomain/front/document.form.php

I see that the folder is created in GLPI structure, the file are sended to this folder but the error is when the document.form is inserting DB informations because this document is not "saved" on DB but is present in /var/www/glpi/files/xpto....

DonutsNL commented 1 year ago

Hi ilicilho,

What glpi and phpsaml versions are you using? Are you enforcing phpSaml?

ilicilho commented 1 year ago

Hi @DonutsNL The phpSAML is 1.1.3 and GLPI is 10.0.6. I´m not enforcing phpSAML

DonutsNL commented 1 year ago

Hi Ilicilho,

Please consider updating to the latest version: https://github.com/derricksmith/phpsaml/releases/tag/1.2.1

ilicilho commented 1 year ago

Thanks @DonutsNL , I´ll do this.