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

Closed
opened 2026-03-22 13:55:01 -05:00 by GiteaMirror · 2 comments
Owner

Originally created by @xpegenaute on GitHub (Feb 18, 2025).

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

Originally created by @xpegenaute on GitHub (Feb 18, 2025). ### 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](https://github.com/wneessen/go-mail/blob/b90943393fea45bf432d17de36af9ec810a2e57b/client.go#L675). 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_
Author
Owner

@kolaente commented on GitHub (Feb 21, 2025):

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?

@kolaente commented on GitHub (Feb 21, 2025): 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?
Author
Owner

@kolaente commented on GitHub (Jun 24, 2025):

Closing as inactive, please ping or open a new issue with relevant information if you still have this problem.

@kolaente commented on GitHub (Jun 24, 2025): Closing as inactive, please ping or open a new issue with relevant information if you still have this problem.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/vikunja#2079