derricksmith / phpsaml

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

Does not save changes #79

Closed gustavo1321 closed 2 years ago

gustavo1321 commented 2 years ago

Hello, thank you all for the input, I have a question.. has it happened to them that at the time of saving the configuration, return to central and do not save the changes? I was doing some research and something appeared regarding a table, but I checked and the table exists, the strange thing is that it does not report logs of any error.

derricksmith commented 2 years ago

Version?

gustavo1321 commented 2 years ago

Version 9.5.7, I did an update to 9.5.4

icttpzgithub commented 2 years ago

Hi Derrick, I seem to be having the same problem. I fill in the form (config.php) and click update at the bottom, but it just returns me back to the blank form, nothing has been saved. Any ideas? I've tried with both FF and chrome phpsaml1.2.0; glpi 10.0.0; Ubuntu 20.04 LTS (I know I'm using unsupported versions) Kind regards DP

icttpzgithub commented 2 years ago

If I manually create the entry in the table INSERT INTO glpi_plugin_phpsaml_configs (id,version, enforced, strict, debug, jit, saml_sp_certificate, saml_sp_certificate_key, saml_sp_nameid_format, saml_idp_entity_id, saml_idp_single_sign_on_service, saml_idp_single_logout_service, saml_idp_certificate, requested_authn_context, requested_authn_context_comparison) -> VALUES ('1', '1.2.0', '0', '1','0','1', '', '', '', .blah, blah, blah. Then afterwards modifying the values via the config.php is working fine.

derricksmith commented 2 years ago

The new version puts all the phpsaml logs into a file called phosaml in the _logs directory of glpi. Can you upload that file?

icttpzgithub commented 2 years ago

sure, here you go:-

phpsaml.log

derricksmith commented 2 years ago

Suggest removing plugin and installing latest version 1.2.0 from Github

brunoMichelFromFrance commented 2 years ago

Hello derrick, I have the same problem, keycloack saml, phpsaml 1.2.0 and glpi 10.0.2. I downloaded phpsaml from the github 1.2.0 release and modified the plugin_phpsaml_check_prerequisites function to make installable. My phpsaml.log is the same as icttpzgithub

edit: It's the same if I clone directly the 1.2.1 branch

derricksmith commented 2 years ago

Can you check the database and let me know what columns exist in the phpsaml plugin table?

jviadair commented 2 years ago

Hi,

I have the same problem. I cannot setup the phpSAML settings via the GUI, I inserted them via SQL

This is the name of columns exist in the phpsaml plugin table: id version enforced strict debug jit saml_sp_certificate saml_sp_certificate_key saml_sp_nameid_format saml_idp_entity_id saml_idp_single_sign_on_service saml_idp_single_logout_service saml_idp_certificate requested_authn_context requested_authn_context_comparison saml_security_nameidencrypted saml_security_authnrequestssigned saml_security_logoutrequestsigned saml_security_logoutresponsesigned

WilliamDR35 commented 2 years ago

Hello, i'm having the same issue. Can someone detail how to fix it please ?

derricksmith commented 2 years ago

Issue was resolved in #106

Can you confirm

WilliamDR35 commented 2 years ago

It's working, thank you !