-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Revert "change mininum TLS version for tag to digest resolution (#13886)" #13956
Conversation
…ive#13886)" This reverts commit 171cecd.
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: kahirokunn The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #13956 +/- ##
=======================================
Coverage 86.25% 86.25%
=======================================
Files 199 199
Lines 14749 14749
=======================================
Hits 12722 12722
- Misses 1726 1727 +1
+ Partials 301 300 -1
☔ View full report in Codecov by Sentry. |
/lgtm Could you change the subject of this PR to make it more clear? I think it is okay to use the commit message |
@nak3 done. how about now? |
Thank you! I am good to merge this. Let's wait for other reviewers in the Europe/US time. |
I'm going to close htis PR since I'm making this property configurable via an env var in this PR - #13962 I also reverted the default back to 1.2. |
This reverts commit 171cecd.
Fixes #13937
Proposed Changes
Release Note