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

Implement LOGIN auth #1

Closed
wants to merge 1 commit into from
Closed

Implement LOGIN auth #1

wants to merge 1 commit into from

Conversation

jerbob92
Copy link

I recently came across an SMTP server that supported LOGIN auth but not PLAIN.

LOGIN and PLAIN are almost the same, PLAIN just sends everything in 1 command, LOGIN sends seperate commands for the auth/username/password, so I decided to implement it in the same auth logic.

@xhit
Copy link
Owner

xhit commented Dec 13, 2019

Thanks for pull request!

I'm going to review the change in 24 hours or least because I'm traveling.

@xhit
Copy link
Owner

xhit commented Dec 17, 2019

In version 2.3.0 was added this feature, also CRAM-MD5 Authentication.

For this, in SMTPServer struct obtained by NewSMTPClient() function, the Authentication variable should be an authType: AuthPlain, AuthLogin, AuthCRAMMD5

If was not specified, default is AuthPlain.

Example:
server.Authentication = mail.AuthLogin

Thanks!, your code help me with the LOGIN Auth.

Also Go Modules compatibility was added.

Thanks for pull request!

@xhit xhit closed this Dec 17, 2019
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

Successfully merging this pull request may close these issues.

None yet

2 participants