quantranhong1999 opened a new pull request #810: URL: https://github.com/apache/james-project/pull/810
WIP: reconsider logic and improve code quality (reduce some if else if it can be,...) As the code stands now, when client does not using TLS connect (server requireSSL) the sever does not advertise any AUTH types: ``` 250-hp-quanth Hello localhost [192.168.42.103]) 250-PIPELINING 250-ENHANCEDSTATUSCODES 250-8BITMIME 250 STARTTLS ``` And only return available auth when client executeTLS: ``` 250-hp-quanth Hello localhost [192.168.42.103]) 250-AUTH OAUTHBEARER 250-AUTH=OAUTHBEARER 250-PIPELINING 250-ENHANCEDSTATUSCODES 250 8BITMIME ``` Is this a correct behavior? Should we return Auth even when client does not use TLS? -- 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]
