Skip to content
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

Required Authentication #13

Open
drkhunter opened this issue Feb 6, 2018 · 0 comments
Open

Required Authentication #13

drkhunter opened this issue Feb 6, 2018 · 0 comments

Comments

@drkhunter
Copy link

I've been looking into requiring authentication as per RFC 4954; returning 530 5.7.0 Authentication required on commands other than AUTH, EHLO, HELO, NOOP, RSET, or QUIT.

To do this I've added an "authenticated" status which gets set to true inside the sendAuthSuccessResponse() method (which feels wrong) and checked for this status before running any of the commands requiring auth.

As it stands in 0.7 if the client does not attempt an AUTH it will just accept any of the other commands.

My next question is in the TRIGGER_NEW_MAIL event, how would one access the current authenticated user, for example to save the email to a database against that user.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant