-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
SAMBA Active Directory password change #8503
Closed
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Added a new utility method: rcube_utils::get_input_string()
* Fix dovecot passwd-file handler * Set new password hash correctly * If password hashing failes during password change request, abort trying to change the password. * Escaping hashed password is not needed and actually breaks if 'password_dovecotpw_with_method' is set to true (escaping the {METHOD} in the passwd file)
The old sound doesn't fit the role of a notification very well. The current sound is more befitting to the objective of notify the user and brings a simple and modern semblance to the project. Signed-off-by: Renan Andrade Torrente <renan.torrente@hotmail.com> Signed-off-by: Igor Matheus Andrade Torrente <igormtorrente@gmail.com>
[skip ci]
The usual dialog padding is 28px in Elastic
When printing a message with a large <blockquote> (e.g. a reply to a long thread), Firefox tries to avoid breaking the page within the blockquote so may add a page break before it, causing the unquoted part of the message to appear on a mostly-blank page.
Even if we do not support it, the fix is trivial and does not make the code worse.
Simplify and unify has_html_part() and has_text_part() methods.
[skip ci]
This can be used by the Docker images where we don't know whether the configured database is already initialized or not.
… structures with a single part (#8458)
…ith no attachments (#8413)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Delete this, I made the wrong request. Sorry.
I've been working to get password changing with a SAMBA AD backend to work with any of the ldap-drivers. Turns out that the ldap_exop_passwd() function does not support SAMBA AD backend:
nextcloud/ldap_write_support#128
I've written a new driver replicating the ldap_exop driver's functionality, which replaces the ldap_exop_passwd() call for ldap_mod_replace() with the appropriate arguments. Maybe it could be included in the official release?