Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport 2.11] [Backport 2.x] Improve wording for TLS-related options #121

Merged
merged 1 commit into from
Sep 28, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 61ceae1 from #119

When configuring a SMTP sender, the user is prompted for an "Encryption
method" and given a drop-down list to choose from.  The list currently
contain "SSL", "TLS" and "None" which is a bit confusing given that TLS
is built on deprecated SSL and both refer to the same thing.

In this list, "SSL" actually mean TLS, and "TLS" actually mean StartTLS
(aka Opportunistic TLS).

Adjust wording to wake this more obvious.  The new wording match the one
used in Thunderbird when configuring connection security.

(cherry picked from commit 2506331)

Signed-off-by: Romain Tartière <romain@blogreen.org>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
(cherry picked from commit 61ceae1)
@codecov
Copy link

codecov bot commented Sep 27, 2023

Codecov Report

Merging #121 (63129e8) into 2.11 (7ccdb06) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             2.11     #121   +/-   ##
=======================================
  Coverage   86.57%   86.57%           
=======================================
  Files          52       52           
  Lines        1497     1497           
  Branches      373      373           
=======================================
  Hits         1296     1296           
  Misses        198      198           
  Partials        3        3           
Flag Coverage Δ
dashboards-notifications 86.57% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@yuye-aws yuye-aws merged commit 689ea14 into 2.11 Sep 28, 2023
6 of 9 checks passed
@github-actions github-actions bot deleted the backport/backport-119-to-2.11 branch September 28, 2023 02:29
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.

2 participants