search

DonutsNL / glpisaml

I moved to: https://codeberg.org/QuinQuies This plugin is a full rewrite of Derrick Smith's initial SAML plugin for GLPI10+. Its redesigned to be compatible with GLPI10+, PSR, Composer, Support multiple saml idp's, user right rules and more.
https://discord.gg/yKZB7VQUk6
GNU General Public License v3.0
10 stars 0 forks source link

Not saved configuration. #11

Closed wmeneses closed 6 months ago

wmeneses commented 7 months ago

Thank you for continuing this project,

I have tried to test it but when I save I get this error "Error: Unable to add new GlpiSaml configuration!!! Leaving a line without text in the configuration, any suggestions?

l-2-m commented 6 months ago

Hi I have the same problem

DonutsNL commented 6 months ago

be aware that this version is still being developed. If you are testing m, thanks for helping 👍🏻Please add screenshots showing the error for my understanding.

If you need a functioning version. Please use the phpsaml repo.

Rgrds,

wmeneses commented 6 months ago

be aware that this version is still being developed. If you are testing m, thanks for helping 👍🏻Please add screenshots showing the error for my understanding.

If you need a functioning version. Please use the phpsaml repo.

Rgrds,

Thanks, I tried the phpsaml version and now it works fine.

DonutsNL commented 6 months ago

The ConfigEntity object was normalizing the configuration. As a result the ID, CREATE_DATE, and MOD_DATE fields where passed as NULL to the COMMONDBTM object for insertion. This is invalid because the config doesnt exist yet and we are relying on the database its AUTO_INCREMENT, and DEFAULT CURRENT_TIMESTAMP functions to auto fill these required fields on first insertion. I modified the SamlConfig->getFieldsForDB() to ignore provided fields and filter them out.

image

Tested insertion of new configurations and it now works as expected.