quantranhong1999 opened a new pull request, #3088:
URL: https://github.com/apache/james-project/pull/3088
Today, the SASL mechanisms being configured automatically by default, that
is great, and it works correctly.
However, for SMTP, if explicitly configure the SASL mechanisms via
`auth.saslMechanisms`, we may have 2 issues:
1. (SMTP) `LoginSaslMechanismFactory` likely can not be explicitly configured
If `auth.saslMechanisms` is absent, we load `LoginSaslMechanismFactory`
correctly by default though.
2. `PlainSaslMechanismFactory` explicitly configured: likely won't honor the
IGNORE_REQUIRE_SSL_CONFIGURATION for SMTP (`GuiceLoader` likely picks the
default `PlainSaslMechanismFactory` constructor and wont diferrentiate this
honor requireSSL difference between IMAP and SMTP)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
