-
-
Notifications
You must be signed in to change notification settings - Fork 32.1k
gh-123954: proposal for improving logic for setting SSL exceptions #128391
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
Draft
picnixz
wants to merge
39
commits into
python:main
Choose a base branch
from
picnixz:perf/ssl/error-123954
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from 1 commit
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
da3968c
introduce semantic types for SSL packed errors
picnixz 704e6e3
correct error deduction in `fill_and_set_sslerror` and `_setSSLError`
picnixz 864339a
pre-initialize default exception attributes
picnixz 87a501d
extract logic for fetching library and reason
picnixz 7cb16c0
extract logic for formatting error message
picnixz 4bd14a0
extract logic for building simple error message
picnixz c29f0f1
extract logic for building SSL verification exceptions
picnixz d8de992
upgrade logic for building SSL exceptions
picnixz ba1bf20
fix various issues
picnixz 11ee17c
use `PyOS_snprintf` instead of `PyUnicode_FromFormat`
picnixz 45c7a1e
fix Windows compilation
picnixz 4362dae
Apply suggestions from code review
picnixz 31ce5f0
Update Modules/_ssl.c
picnixz 2fbcae2
post-merge
picnixz fd7d3ec
use `_Py_hashtable_t` for `err_codes_to_names`
picnixz 7772113
use `_Py_hashtable_t` for `lib_codes_to_names`
picnixz 4a2ed1d
post-merge
picnixz 630c2b5
fix compilation?
picnixz 98c8b99
attempt various things
picnixz 133ec9f
try even more things!
picnixz fb5d800
:@
picnixz d6a0529
:@@@@@@@@@@@@@@
picnixz bb043b2
skip duplicated entries if they are correct
picnixz f836684
log more stuff
picnixz db3ed1c
change hash function key comparator?
picnixz 7a198d4
revert macro
picnixz 93a7a9b
full debug
picnixz 95e1021
remove un-necessary specializations
picnixz b8f0cce
select when to use verbose SSL errors
picnixz 689568f
reduce the number of format units
picnixz 5583aa5
optimize `format_ssl_error_message`
picnixz a0ab1c2
temporarily patch CI
picnixz 6b38769
fix assertions
picnixz 3ecba48
now fix assertions correctly
picnixz d7f1ab1
use ImportWarning insteaad of RuntimeWarning
picnixz 32f7cba
clear exception after emitting the warning
picnixz a88ec91
fix error path when formatting message
picnixz a9a6a50
Merge remote-tracking branch 'upstream/main' into perf/ssl/error-123954
picnixz 68f8bc7
include hexadecimal error code value when reporting unknown error
picnixz File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Apply suggestions from code review
- Loading branch information
commit 4362dae668036602cafba4265c39a79034f310a4
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.