search

sonicnkt / glauth-ui

Glauth management ui created with python/flask
MIT License
85 stars 19 forks source link

Improvements to auto-generated glauth config #18

Closed sharatbala closed 2 years ago

sharatbala commented 2 years ago

The original version generated config files that would not return valid results for ldapsearch or Authelia. The reasons were the use of the deprecated "unixid" field as well as the missing behavior attribute. This change sets up default behavior to allow all users to search. Ideally the UI would be updated with a checkbox or something like that per user which would then add a [behaviour] section for each user

sonicnkt commented 2 years ago

Thanks for the contribution.. There is allreadz a PR for the changes that need to be implemented for supporting glauth 2.x (see #17 ) I have merged that PR into a new branch, but i havent fully figured out yet how to migrate the existing databases.

sonicnkt commented 2 years ago

Okay im still having issues with the db migration if i also rename the columns in the db itself. i will go with your more simple approach first (only change the names in the ui and generated config file)

I will merge this by hand tho because of some conflicts but thanks for the contribution! Im really out of touch with git after not using it for some time. but i really want to get this going again with the up to date version of glauth