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

New Issue

GiteaMirror · 2026-04-23T05:41:30-05:00

GiteaMirror commented

2026-04-23 05:41:30 -05:00

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!

GiteaMirror added the wontfix label 2026-04-23 05:41:30 -05:00

GiteaMirror closed this issue

2026-04-23 05:41:32 -05:00

GiteaMirror commented

2026-04-23 05:41:33 -05:00

@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)

@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.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/vaultwarden#13982