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

gomail library unmaintained #18664

Closed
fnetX opened this issue Feb 7, 2022 · 11 comments · Fixed by #32682
Closed

gomail library unmaintained #18664

fnetX opened this issue Feb 7, 2022 · 11 comments · Fixed by #32682
Labels
type/upstream This is an issue in one of Gitea's dependencies and should be reported there

Comments

@fnetX
Copy link
Contributor

fnetX commented Feb 7, 2022

The gomail library used by Gitea is out of date. Transfer from me from the channel (Matrix)

Gitea seems to use https://github.com/go-gomail/gomail which is unmaintained since 2016, and the "actively maintained" fork appears unmaintained, too: https://github.com/go-mail/mail/
Do you consider this an issue for you? I looked at some Gitea issues, and some cases like different timeouts / retries is also a feature request to both frameworks, but apparently never implemented. Does a switch make sense, or is the current state perfectly fine?


Discussion already started here:

FYI

Maybe we should maintain gomail by ourselves, or switch to another one. Then we may not need hacky code.

Some active forks:

Originally posted by @wxiaoguang in #18075 (comment)

@wxiaoguang
Copy link
Contributor

wxiaoguang commented Feb 8, 2022

FYI, there were more comments.

  1. These active forks seem not ideal. Add option to convert CRLF to LF line endings for sendmail #18075 (comment)
  2. IIRC many contributors accepted to maintain the gomail by ourselves since it's not complex.

@lunny
Copy link
Member

lunny commented Feb 8, 2022

How about convert to https://github.com/xhit/go-simple-mail .

@noerw noerw added the type/upstream This is an issue in one of Gitea's dependencies and should be reported there label Mar 12, 2022
@fredmenez
Copy link

FYI the maintainer of go-gomail/gomail has left this world, details in go-gomail/gomail#182.

@techknowlogick
Copy link
Member

@fredmenez thank you for sharing this news with us ❤️ 🕯️

@lunny
Copy link
Member

lunny commented Nov 23, 2024

Maybe https://github.com/wneessen/go-mail is a better choice.

@Neustradamus
Copy link

@lunny: Thanks for your reactivity!
Yes totally! :)

@lunny
Copy link
Member

lunny commented Nov 23, 2024

First step #32618 to move to the new package.

@Neustradamus
Copy link

@lunny: Thanks for your quick PR!

lafriks pushed a commit that referenced this issue Nov 30, 2024
Move all mail sender related codes into a sub package of
services/mailer. Just move, no code change.
Then we just have dependencies on go-mail package in the new sub
package. We can use other package to replace it because it's
unmaintainable. ref #18664
@Neustradamus
Copy link

@Neustradamus
Copy link

@lunny: Good job, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/upstream This is an issue in one of Gitea's dependencies and should be reported there
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants