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

Avoid using NOOP when creating the go-mail client #409

Open
xpegenaute opened this issue Feb 18, 2025 · 1 comment
Open

Avoid using NOOP when creating the go-mail client #409

xpegenaute opened this issue Feb 18, 2025 · 1 comment

Comments

@xpegenaute
Copy link

Description

Hi All,

in my smtp context I am having this error when I am trying to connect to the smtp server:
Error during connect to smtp server: failed to authenticate: not connected to SMTP server

Debugging the connection I found that it is related with the use of the operation "NOOP" by the go-mail client. Even though it should usually work, some times you can find some issues, like my case.

Is it possible to add another option into de go-mail options in order to include the mail.WithoutNoop() option ?

Regards

Vikunja Version

0.24.6

Browser and version

No response

Can you reproduce the bug on the Vikunja demo site?

No

Screenshots

No response

@kolaente
Copy link
Member

I'd love a PR with a change like this, but I'm unsure about the implications. Would it hurt to include that option for everyone or should it be a configurable option?

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

2 participants