Skip to content

Replace usage of the obscure PEM_read_bio_X509_AUX with the more standard PEM_read_bio_X509 #15303

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

Merged
merged 1 commit into from
Aug 15, 2019

Conversation

alex
Copy link
Member

@alex alex commented Aug 15, 2019

X509_AUX is an odd, note widely used, OpenSSL extension to the X509 file format. This function doesn't actually use any of the extra metadata that it parses, so just use the standard API.

Automerge-Triggered-By: @tiran

@miss-islington
Copy link
Contributor

@alex: Status check is done, and it's a success ✅ .

@miss-islington miss-islington merged commit 40dad95 into master Aug 15, 2019
@miss-islington miss-islington deleted the alex-patch-1 branch August 15, 2019 12:31
@miss-islington
Copy link
Contributor

Thanks @alex for the PR 🌮🎉.. I'm working now to backport this PR to: 3.7, 3.8.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 15, 2019
…dard PEM_read_bio_X509 (pythonGH-15303)

X509_AUX is an odd, note widely used, OpenSSL extension to the X509 file format. This function doesn't actually use any of the extra metadata that it parses, so just use the standard API.

Automerge-Triggered-By: @tiran
(cherry picked from commit 40dad95)

Co-authored-by: Alex Gaynor <[email protected]>
@bedevere-bot
Copy link

GH-15304 is a backport of this pull request to the 3.8 branch.

@bedevere-bot
Copy link

GH-15305 is a backport of this pull request to the 3.7 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 15, 2019
…dard PEM_read_bio_X509 (pythonGH-15303)

X509_AUX is an odd, note widely used, OpenSSL extension to the X509 file format. This function doesn't actually use any of the extra metadata that it parses, so just use the standard API.

Automerge-Triggered-By: @tiran
(cherry picked from commit 40dad95)

Co-authored-by: Alex Gaynor <[email protected]>
miss-islington added a commit that referenced this pull request Aug 15, 2019
…e standard PEM_read_bio_X509 (GH-15303) (GH-15304)

X509_AUX is an odd, note widely used, OpenSSL extension to the X509 file format. This function doesn't actually use any of the extra metadata that it parses, so just use the standard API.

Automerge-Triggered-By: @tiran
(cherry picked from commit 40dad95)


Co-authored-by: Alex Gaynor <[email protected]>

Automerge-Triggered-By: @tiran
miss-islington added a commit that referenced this pull request Aug 15, 2019
…dard PEM_read_bio_X509 (GH-15303)

X509_AUX is an odd, note widely used, OpenSSL extension to the X509 file format. This function doesn't actually use any of the extra metadata that it parses, so just use the standard API.

Automerge-Triggered-By: @tiran
(cherry picked from commit 40dad95)

Co-authored-by: Alex Gaynor <[email protected]>
lisroach pushed a commit to lisroach/cpython that referenced this pull request Sep 10, 2019
…dard PEM_read_bio_X509 (pythonGH-15303)

X509_AUX is an odd, note widely used, OpenSSL extension to the X509 file format. This function doesn't actually use any of the extra metadata that it parses, so just use the standard API.

Automerge-Triggered-By: @tiran
DinoV pushed a commit to DinoV/cpython that referenced this pull request Jan 14, 2020
…dard PEM_read_bio_X509 (pythonGH-15303)

X509_AUX is an odd, note widely used, OpenSSL extension to the X509 file format. This function doesn't actually use any of the extra metadata that it parses, so just use the standard API.

Automerge-Triggered-By: @tiran
websurfer5 pushed a commit to websurfer5/cpython that referenced this pull request Jul 20, 2020
…dard PEM_read_bio_X509 (pythonGH-15303)

X509_AUX is an odd, note widely used, OpenSSL extension to the X509 file format. This function doesn't actually use any of the extra metadata that it parses, so just use the standard API.

Automerge-Triggered-By: @tiran
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants