You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The return value of ldap_mod_replace is not checked when setting the Mail. Would be nice to check for that and to throw an exception in case of failure.
This is not only a theoretical concern: I am currently trying to enforce unique E-Mails through LDAP and setting a non-unique one just silently fails at the moment.
The text was updated successfully, but these errors were encountered:
That won't be enough. Currently, the server code sets the email address (if allowed) in the database, then emits an Event that it was changed. A cancellable Pre-Event is required. Could you open an issue on the server repo? Likely we meet there again :)
The return value of
ldap_mod_replace
is not checked when setting the Mail. Would be nice to check for that and to throw an exception in case of failure.This is not only a theoretical concern: I am currently trying to enforce unique E-Mails through LDAP and setting a non-unique one just silently fails at the moment.
The text was updated successfully, but these errors were encountered: