Skip to content

Require target.from as fallback, add replyTo support with sensible defaults#13

Open
DrSleez wants to merge 1 commit intoFeuerhamster:masterfrom
FiduxIT:reply_to
Open

Require target.from as fallback, add replyTo support with sensible defaults#13
DrSleez wants to merge 1 commit intoFeuerhamster:masterfrom
FiduxIT:reply_to

Conversation

@DrSleez
Copy link
Copy Markdown

@DrSleez DrSleez commented Sep 25, 2025

  • Added support for replyTo on incoming requests.
  • Made target.from required and used it as a fallback when no from is provided in the request.
  • If replyTo is not provided, it defaults to the effective from (field from if present, otherwise target.from).
  • Request field from continues to override target.from.
  • Updated README to document required/optional fields and new defaults.

Why

  • We needed replyTo and noticed there was an open issue requesting the same feature.
  • Making target.from required ensures a reliable fallback and avoids sending emails without a valid sender.

Behavior

  • from: request field overrides; otherwise fallback to target.from (required).
  • replyTo: optional; defaults to the effective from when not supplied.

Docs

  • README updated: targets and request fields grouped into Required vs Optional; replyTo behavior documented.

Migration

  • Existing targets must now include from (required). Add a default like no-reply@example.com if missing.

Closes

Closes #9

@Feuerhamster
Copy link
Copy Markdown
Owner

Thank you, but your PR messed up some spacings. Can you fix it? I will merge it then.

@DrSleez
Copy link
Copy Markdown
Author

DrSleez commented Sep 29, 2025

Hey, sure could you tell me which spacings you mean? It was probably my prettier addon which messed something up.

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

Successfully merging this pull request may close these issues.

Allow reply-to to be set via form

2 participants