Server error when adding user

devilry / trix2

Next generation Trix. Detailed task control and statistics app for better learning outcome.

BSD 3-Clause "New" or "Revised" License

2 stars 3 forks source link

Server error when adding user #61

Closed torgeirl closed 5 years ago

torgeirl commented 6 years ago

Add user leads to a 500 server error. I'm guessing this could be related to LDAP (and haven't had a chance to test it without LDAP).

Add new user requires an email address while ldap_to_django_username() / django_to_ldap_username() assumes django see a regular username. Perhaps that's the problem.

Elthan commented 6 years ago

Looks like it is related to LDAP as I could not recreate the issue on a test server. Some logic to fix this could be implemented, but will not be a high priority since it should not be a critical issue (new users simply need to log in to be created when using LDAP).

torgeirl commented 6 years ago

I think a solution should just be to deactivate the Add user option when AUTHENTICATION_BACKENDS doesn't contain a non-LDAP backend. I'm unfamiliar with how that would be implemented, but believe something similar is done in Devilry.

torgeirl commented 5 years ago

It seems the issue was an old LDAP config. The issue is no longer present with the current LDAP config.