search

eXist-db / exist

eXist Native XML Database and Application Platform
https://exist-db.org
GNU Lesser General Public License v2.1
428 stars 179 forks source link

sm:get-group-managers #761

Open claudius108 opened 9 years ago

claudius108 commented 9 years ago

Hi,

With eXist 3.0, sm:get-group-managers() does not return all the group managers. If I am looking into the group file itself, and set the group managers with sm:add-group-manager(), the function sm:get-group-managers() returns correctly the group managers. Although, the correct result does not persist if I restart eXist.

adamretter commented 9 years ago

I started investigating the issue here - https://github.com/adamretter/exist/tree/configurator-group-manager

I can reproduce the issue with this test case - https://github.com/adamretter/exist/blob/configurator-group-manager/test/src/org/exist/security/SecurityManagerRoundtripTest.java#L229

Unfortunately the issue is in the Configurator and it appears to be non-trivial to fix. Ideally we should replace the Configurator https://github.com/eXist-db/exist/issues/578 but that is rather a large piece of work.

claudius108 commented 9 years ago

Thanks for the insights, Adam.

duncdrum commented 2 years ago

this still seems to be broken

duncdrum commented 2 years ago

as for the lack of persistence when using sm:add-group-manger() i noticed that it changes the <metadata> key in the group's xml file from key="http://exist-db.org/security/description" to key="metadata"

groups with 

<metadata key="metadata"/>

will be correctly returned by sm:get-group-managers()

groups with 

<metadata key="http://exist-db.org/security/description"/>

will not