felixauringer commented on code in PR #2773:
URL: https://github.com/apache/james-project/pull/2773#discussion_r2510084865
##########
protocols/managesieve/src/main/java/org/apache/james/managesieve/core/PlainAuthenticationProcessor.java:
##########
@@ -50,7 +50,7 @@ public PlainAuthenticationProcessor(UsersRepository
usersRepository) {
@Override
public String initialServerResponse(Session session) {
- return "+ \"\"";
+ return "\"\"\r\nOK";
Review Comment:
Okay, I found the [PLAIN SASL
RFC](https://datatracker.ietf.org/doc/html/rfc4616#section-4) from where your
previous server response seems to come from. I really do think that it
conflicts with the managesieve RFC (5804) but I am fine with keeping the old
syntax then.
--
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]
