[GH-ISSUE #2225] Env-Variable - Renaming SMTP_EXPLICIT_TLS #1194

Closed
opened 2026-03-03 02:07:03 +03:00 by kerem · 1 comment
Owner

Originally created by @sando38 on GitHub (Jan 11, 2022).
Original GitHub issue: https://github.com/dani-garcia/vaultwarden/issues/2225

Hello there,

reporting this, because I missinterpreted it. According to RFC8314 (https://datatracker.ietf.org/doc/html/rfc8314#section-3.3) implicit TLS is used for port 465.
However, to force tls with environment variables in the smtp settings, I have to set:

SMTP_EXPLICIT_TLS=true -> which would actually imply STARTTLS.

So, either I suggest renaming the environment variable into:

SMTP_IMPLICIT_TLS or
SMTP_FORCE_TLS

to avoid confusion.

NB: Thanks for your great service with providing vaultwarden. It is a great tool.

Wishing a nice day!

Originally created by @sando38 on GitHub (Jan 11, 2022). Original GitHub issue: https://github.com/dani-garcia/vaultwarden/issues/2225 Hello there, reporting this, because I missinterpreted it. According to RFC8314 (https://datatracker.ietf.org/doc/html/rfc8314#section-3.3) implicit TLS is used for port 465. However, to force tls with environment variables in the smtp settings, I have to set: SMTP_EXPLICIT_TLS=true -> which would actually imply STARTTLS. So, either I suggest renaming the environment variable into: SMTP_IMPLICIT_TLS or SMTP_FORCE_TLS to avoid confusion. NB: Thanks for your great service with providing vaultwarden. It is a great tool. Wishing a nice day!
kerem 2026-03-03 02:07:03 +03:00
  • closed this issue
  • added the
    wontfix
    label
Author
Owner

@BlackDex commented on GitHub (Jan 11, 2022):

This will be a wont fix, since this would lead to a lot of invalid configurations. A lot of people have this configured using that variable and we would not like to break those. Also, adding a second variable and keeping the current one will probably lead to confusion or also some weird issues.

There is already a mentioning of the wrongly named variable here: https://github.com/dani-garcia/vaultwarden/blob/main/.env.template#L335

Also see #851 (which this ticket is a duplicate of)

<!-- gh-comment-id:1009702441 --> @BlackDex commented on GitHub (Jan 11, 2022): This will be a wont fix, since this would lead to a lot of invalid configurations. A lot of people have this configured using that variable and we would not like to break those. Also, adding a second variable and keeping the current one will probably lead to confusion or also some weird issues. There is already a mentioning of the wrongly named variable here: https://github.com/dani-garcia/vaultwarden/blob/main/.env.template#L335 Also see #851 (which this ticket is a duplicate of)
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/vaultwarden#1194
No description provided.