◐ Shell
reader mode source ↗
Skip to content

[3.12] gh-137586: Open external osascript program with absolute path (GH-137584)#148175

Merged
hugovk merged 1 commit into
python:3.12from
miss-islington:backport-a0c57a8-3.12
Apr 7, 2026
Merged

[3.12] gh-137586: Open external osascript program with absolute path (GH-137584)#148175
hugovk merged 1 commit into
python:3.12from
miss-islington:backport-a0c57a8-3.12

Conversation

@miss-islington

@miss-islington miss-islington commented Apr 6, 2026

Copy link
Copy Markdown
Contributor

Open web browser with absolute path

On macOS, web browsers are opened via popen calling osascript. However,
if a user has a colliding osascript executable earlier in their PATH,
this may fail or cause unwanted behaviour.

Depending on one's environment or level of paranoia, this may be considered a security vulnerability.
(cherry picked from commit a0c57a8)

Co-authored-by: Fionn 1897918+fionn@users.noreply.github.com
Co-authored-by: Hugo van Kemenade 1324225+hugovk@users.noreply.github.com

…ythonGH-137584)

Open web browser with absolute path

On macOS, web browsers are opened via popen calling osascript. However,
if a user has a colliding osascript executable earlier in their PATH,
this may fail or cause unwanted behaviour.

Depending on one's environment or level of paranoia, this may be considered a security vulnerability.
(cherry picked from commit a0c57a8)

Co-authored-by: Fionn <1897918+fionn@users.noreply.github.com>
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
@python-cla-bot

python-cla-bot Bot commented Apr 6, 2026

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.

CLA signed

@bedevere-app bedevere-app Bot added the type-security A security issue label Apr 6, 2026
@hugovk hugovk enabled auto-merge (squash) April 6, 2026 17:16
Hide details View details @hugovk hugovk merged commit 20b4393 into python:3.12 Apr 7, 2026
29 checks passed
@miss-islington miss-islington deleted the backport-a0c57a8-3.12 branch April 7, 2026 03:35
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.

3 participants