Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check for failure on setting the Mail #164

Open
VincentWo opened this issue Jun 3, 2020 · 1 comment
Open

Check for failure on setting the Mail #164

VincentWo opened this issue Jun 3, 2020 · 1 comment

Comments

@VincentWo
Copy link

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.

@blizzz
Copy link
Member

blizzz commented Jun 3, 2020

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 :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants