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

Deprecate ssl common name #2738

Merged
merged 1 commit into from
Aug 30, 2022

Conversation

dlm6693
Copy link
Contributor

@dlm6693 dlm6693 commented Aug 16, 2022

Now that aws/aws-cli#7114 is close to full approval, this should be re-opened and merged to deprecate the SSL common name paradigm in botocore. This is a copy of https://github.com/boto/botocore/pull/2704/files which had to be rolled back because of unforeseen issues it caused in aws-cli v1.

@dlm6693 dlm6693 force-pushed the deprecate-ssl-common-name branch 2 times, most recently from da55941 to 544979c Compare August 16, 2022 22:09
@codecov-commenter
Copy link

codecov-commenter commented Aug 16, 2022

Codecov Report

Merging #2738 (d9fc6db) into develop (474e7a2) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop    #2738   +/-   ##
========================================
  Coverage    95.25%   95.26%           
========================================
  Files           60       60           
  Lines        12278    12285    +7     
========================================
+ Hits         11696    11703    +7     
  Misses         582      582           
Impacted Files Coverage Δ
botocore/client.py 98.74% <100.00%> (+0.01%) ⬆️

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

@dlm6693 dlm6693 force-pushed the deprecate-ssl-common-name branch 4 times, most recently from d276304 to aafb5ee Compare August 17, 2022 14:05
@dlm6693 dlm6693 force-pushed the deprecate-ssl-common-name branch from aafb5ee to d9fc6db Compare August 17, 2022 14:08
Copy link
Contributor

@nateprewitt nateprewitt left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@nateprewitt nateprewitt merged commit 3f2350f into boto:develop Aug 30, 2022
@dlm6693 dlm6693 deleted the deprecate-ssl-common-name branch August 30, 2022 15:42
aws-sdk-python-automation added a commit that referenced this pull request Aug 30, 2022
* release-1.27.63:
  Bumping version to 1.27.63
  Update to latest endpoints
  Update to latest models
  Deprecate SSL common name (#2738)
  Convert dataclass to NamedTuple for backwards compatibility (#2748)
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.

3 participants