🐛 Bug Report: SMTP_FROM does not account for mails in format of User <user@example.com> #128

Closed
opened 2025-10-09 16:27:54 +03:00 by OVERLORD · 4 comments
Owner

Originally created by @stavros-k on GitHub.

Originally assigned to: @kmendell on GitHub.

Reproduction steps

Using the above will result in an error like Relaying is not allowed at @

Expected behavior

The above format to work. You can probably use net/mail to ParseAddress().

Actual Behavior

Using the above will result in an error like Relaying is not allowed at @

Version and Environment

v1.6.2

Log Output

No response

Originally created by @stavros-k on GitHub. Originally assigned to: @kmendell on GitHub. ### Reproduction steps Using the above will result in an error like `Relaying is not allowed at @` ### Expected behavior The above format to work. You can probably use net/mail to ParseAddress(). ### Actual Behavior Using the above will result in an error like `Relaying is not allowed at @` ### Version and Environment v1.6.2 ### Log Output _No response_
OVERLORD added the bug label 2025-10-09 16:27:54 +03:00
Author
Owner

@kmendell commented on GitHub:

You should just use the email format, the email should auto append Pocket ID onto it, so the email will come from Pocket ID user@example.com.

@kmendell commented on GitHub: You should just use the email format, the email should auto append Pocket ID onto it, so the email will come from Pocket ID <user@example.com>.
Author
Owner

@kmendell commented on GitHub:

Add the format in 9e52783

@kmendell commented on GitHub: Add the format in [9e52783](https://github.com/pocket-id/website/commit/9e527831f9773dc2e2146a8f29919fca8697af0e)
Author
Owner

@kmendell commented on GitHub:

I can update the docs a little later today to make sure that is more clear :)

@kmendell commented on GitHub: I can update the docs a little later today to make sure that is more clear :)
Author
Owner

@stavros-k commented on GitHub:

Yep figure it out eventually, but error is not intuitive, neither docs mention the correct format.
At least docs might be good to indicate that.

@stavros-k commented on GitHub: Yep figure it out eventually, but error is not intuitive, neither docs mention the correct format. At least docs might be good to indicate that.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/pocket-id-pocket-id-2#128