FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[3.12] gh-127667: refactor and improve `_hashopenssl.c` error branches (#131145) by picnixz · Pull Request #131348 · python/cpython · GitHub

Repository navigation

[3.12] gh-127667: refactor and improve _hashopenssl.c error branches (#131145) - #131348

Merged
picnixz merged 1 commit into
python:3.12from
picnixz:bp-312/ac50ece6cea8745834e4ec0a9617809a51245bfc/refactor-hashlib-127667
Mar 18, 2025
Merged

picnixz merged 1 commit into
python:3.12from
picnixz:bp-312/ac50ece6cea8745834e4ec0a9617809a51245bfc/refactor-hashlib-127667

Conversation

picnixz commented Mar 17, 2025 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Member

Refactor _setException() into different helpers that can be used separately:

  • set_ssl_exception_from_errcode(): set an exception from an explicit SSL error code.
  • raise_ssl_error(): set an exception from the last SSL error code or use a user-defined message.
  • notify_ssl_error_occurred(): same as raise_ssl_error() but with a generic default message.

(cherry-picked from commit ac50ece).

…python#131145)

Refactor `_setException()` into different helpers that can be used separately:

- set_ssl_exception_from_errcode(): set an exception from an explicit SSL error code.
- raise_ssl_error(): set an exception from the last SSL error code or use a user-defined message.
- notify_ssl_error_occurred(): same as raise_ssl_error() but with a generic default message.
picnixz requested a review from gpshead March 17, 2025 10:27
picnixz merged commit 304ef84 into python:3.12 Mar 18, 2025
picnixz deleted the bp-312/ac50ece6cea8745834e4ec0a9617809a51245bfc/refactor-hashlib-127667 branch March 18, 2025 09:26
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
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.

1 participant


Back | FazBrowse Home | New Git URL