Skip to content
This repository was archived by the owner on Jul 11, 2023. It is now read-only.

Conversation

whitneygriffith
Copy link
Contributor

Description: Part of #5282. This removes support for TLSv1_0 and TLSv1_1 from the envoy TLSMaxProtocolVersion options.

Testing done:

Affected area:

Functional Area
New Functionality [ ]
CI System [ ]
CLI Tool [ ]
Certificate Management [ ]
Control Plane [x]
Demo [ ]
Documentation [ ]
Egress [ ]
Ingress [ ]
Install [ ]
Networking [x]
Observability [ ]
Performance [ ]
SMI Policy [ ]
Security [ ]
Sidecar Injection [ ]
Tests [ ]
Upgrade [ ]
Other [ ]

@codecov-commenter
Copy link

codecov-commenter commented Mar 22, 2023

Codecov Report

Merging #5298 (c3f6032) into release-v1.2 (1a9b067) will decrease coverage by 0.02%.
The diff coverage is n/a.

❗ Current head c3f6032 differs from pull request most recent head e57e7d2. Consider uploading reports for the commit e57e7d2 to get more accurate results

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@               Coverage Diff                @@
##           release-v1.2    #5298      +/-   ##
================================================
- Coverage         68.83%   68.81%   -0.02%     
================================================
  Files               210      210              
  Lines             15679    15679              
================================================
- Hits              10792    10789       -3     
- Misses             4836     4839       +3     
  Partials             51       51              
Flag Coverage Δ
unittests 68.81% <ø> (-0.02%) ⬇️

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

see 1 file with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@whitneygriffith whitneygriffith force-pushed the release-v1.2 branch 2 times, most recently from 246058a to 286063f Compare March 27, 2023 19:28
@whitneygriffith whitneygriffith force-pushed the release-v1.2 branch 2 times, most recently from 7a465bd to c2d863d Compare March 27, 2023 21:29
keithmattix
keithmattix previously approved these changes Mar 28, 2023
jaellio
jaellio previously approved these changes Mar 30, 2023
Copy link
Contributor

@jaellio jaellio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, left a few non blocking nits

keithmattix
keithmattix previously approved these changes Mar 30, 2023
…rotocolVersion

Signed-off-by: Whitney Griffith <whitney.griffith16@gmail.com>
@whitneygriffith whitneygriffith requested a review from jaellio March 30, 2023 20:35
@jaellio jaellio merged commit 00fd7e3 into openservicemesh:release-v1.2 Mar 30, 2023
@keithmattix
Copy link
Contributor

@whitneygriffith don't forget to submit a similar PR for the main branch

jaellio pushed a commit to jaellio/osm that referenced this pull request Apr 5, 2023
…rotocolVersion (openservicemesh#5298)

Removes support for TLSv1_0 and TLSv1_1 from the envoy 
TLSMaxProtocolVersion options

Signed-off-by: Whitney Griffith <whitney.griffith16@gmail.com>
jaellio pushed a commit to jaellio/osm that referenced this pull request Apr 5, 2023
…rotocolVersion (openservicemesh#5298)

Removes support for TLSv1_0 and TLSv1_1 from the envoy
TLSMaxProtocolVersion options

Signed-off-by: Whitney Griffith <whitney.griffith16@gmail.com>
Signed-off-by: jaellio <jaellio@microsoft.com>
jaellio pushed a commit to jaellio/osm that referenced this pull request Apr 7, 2023
…rotocolVersion (openservicemesh#5298)

Removes support for TLSv1_0 and TLSv1_1 from the envoy
TLSMaxProtocolVersion options

Signed-off-by: Whitney Griffith <whitney.griffith16@gmail.com>
Signed-off-by: jaellio <jaellio@microsoft.com>
jaellio pushed a commit to jaellio/osm that referenced this pull request Apr 7, 2023
…rotocolVersion (openservicemesh#5298)

Removes support for TLSv1_0 and TLSv1_1 from the envoy
TLSMaxProtocolVersion options

Signed-off-by: Whitney Griffith <whitney.griffith16@gmail.com>
Signed-off-by: jaellio <jaellio@microsoft.com>
jaellio pushed a commit to jaellio/osm that referenced this pull request Apr 11, 2023
…rotocolVersion (openservicemesh#5298)

Removes support for TLSv1_0 and TLSv1_1 from the envoy
TLSMaxProtocolVersion options

Signed-off-by: Whitney Griffith <whitney.griffith16@gmail.com>
Signed-off-by: jaellio <jaellio@microsoft.com>
jaellio pushed a commit that referenced this pull request Apr 11, 2023
…rotocolVersion (#5298)

Removes support for TLSv1_0 and TLSv1_1 from the envoy
TLSMaxProtocolVersion options

Signed-off-by: Whitney Griffith <whitney.griffith16@gmail.com>
Signed-off-by: jaellio <jaellio@microsoft.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants