mirror of
https://github.com/binwiederhier/ntfy.git
synced 2026-05-09 08:26:00 +02:00
[GH-ISSUE #645] [SMTP] Allow unsecured mailservers #480
Labels
No labels
ai-generated
android-app
android-app
android-app
🪲 bug
build
build
dependencies
docs
enhancement
enhancement
🔥 HOT
in-progress 🏃
ios
prio:low
prio:low
pull-request
question
🔒 security
server
server
unified-push
web-app
website
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/ntfy#480
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @sharknoon on GitHub (Mar 2, 2023).
Original GitHub issue: https://github.com/binwiederhier/ntfy/issues/645
Hello,
I have ntfy deployed in a containerized environment with docker. There I have an external mailserver (SMTP) that is only accessible via a docker network. My mailserver has no authentification, because it is not public. Only containers in the same network can access it.
When I leave the field
smtp-sender-user/smtp-sender-passblank, the error is the following:When I comment it out, the error is the following:
This error comes from here.
I would be very happy, if we would allow mailservers without auth. Especially for cases, where the mailserver is exclusive to ntfy by being bundled in the same docker-compose network.
Btw Thank you @binwiederhier for making such a great app and putting so much effort in it, I am loving ntfy :)
Kind regards from the southwest of germany (Stuttgart)
Sharknoon
@binwiederhier commented on GitHub (Mar 3, 2023):
I implemented this in
github.com/binwiederhier/ntfy@90db1283ddSadly I do not have an SMTP server without to test this with, but I'm pretty sure it'll work. If you know how to build ntfy, you can build the latest
mainbranch. If not, I built an amd64 version for you here:@binwiederhier commented on GitHub (Mar 3, 2023):
I'm closing this ticket since I assume it works, If it doesn't, please let me know.
@sharknoon commented on GitHub (Mar 3, 2023):
Thank you so much for your very fast implementation. I was already working on a pull request, made the same changes, but you were faster :) Thanks!
@binwiederhier commented on GitHub (Mar 3, 2023):
I'm sorry I took that from you. It just felt so easy, and I sometimes like doing easy things inbetween all the weird and complicated stuff.