Skip to content

gh-159739: Propagate macOS proxy conversion errors - #159750

Merged
JelleZijlstra merged 1 commit into
python:mainfrom
jelle-openai:jelle/codex/gh-159739-macos-proxy
Oct 20, 2026
Merged

JelleZijlstra merged 1 commit into
python:mainfrom
jelle-openai:jelle/codex/gh-159739-macos-proxy

Conversation

@jelle-openai

@jelle-openai jelle-openai commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

Propagate Python exceptions from Core Foundation string conversion
instead of returning a result with an exception set.

This function (somewhat confusingly) can both return NULL by way of CFStringGetCString, and also return NULL when a Python API call fails.

Propagate Python exceptions from Core Foundation string conversion
instead of returning a result with an exception set.

Backport to 3.15 and 3.14.
@JelleZijlstra JelleZijlstra added needs backport to 3.14 bugs and security fixes needs backport to 3.15 bugs and security fixes labels Oct 19, 2026
@JelleZijlstra
JelleZijlstra enabled auto-merge (squash) October 20, 2026 02:30
@JelleZijlstra
JelleZijlstra merged commit 6044c62 into python:main Oct 20, 2026
62 checks passed
@bedevere-app

bedevere-app Bot commented Oct 20, 2026

Copy link
Copy Markdown

GH-159932 is a backport of this pull request to the 3.14 branch.

@bedevere-app

bedevere-app Bot commented Oct 20, 2026

Copy link
Copy Markdown

GH-159934 is a backport of this pull request to the 3.15 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.15 bugs and security fixes label Oct 20, 2026
JelleZijlstra pushed a commit that referenced this pull request Oct 20, 2026
…#159932)

Propagate Python exceptions from Core Foundation string conversion instead of returning a result with an exception set.

(cherry picked from commit 6044c62)
JelleZijlstra pushed a commit that referenced this pull request Oct 20, 2026
…#159934)

Propagate Python exceptions from Core Foundation string conversion instead of returning a result with an exception set.

(cherry picked from commit 6044c62)
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.

2 participants