-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Fix #443 - RIMAP support #448
Conversation
Signed-off-by: Wolfgang Ocker <weo@reccoware.de>
Signed-off-by: Wolfgang Ocker <weo@reccoware.de>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this PR.
Perhaps we also should add few checks on startup because if rimap
is used, ENABLE_SASLAUTHD
should be set to 1
, no?
@@ -0,0 +1,5 @@ | |||
EHLO mail |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps you should reuse the smpt auth (renaming the file to something more obvious, specifying the user perhaps). Because if we change a password, it should be done in both file (the probably look the same)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, reusing smtp-auth-login.txt makes sense.
Regarding checks of ENABLE_SASLAUTHD
: perhaps we can drop that variable at all and just look at SASLAUTHD_MECHANISMS
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's an option. perhaps LDAP users may have an idea on that specific point.
But I'm currently refactoring tests and I can say that's not obvious when we look at the different combinations with these variables.
…map auth - remove sasl-imap-smtp-auth.txt again Signed-off-by: Wolfgang Ocker <weo@reccoware.de>
Thanks @weo! |
* Add unit tests for docker-mailserver#443 (rimap auth) * Fix docker-mailserver#443 - configure rimap for saslauth * Fix docker-mailserver#443 - reuse smtp-auth-login.txt when testing rimap auth
#443