Skip to content

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

Merged
hugovk merged 1 commit intopython:3.14from
miss-islington:backport-a0c57a8-3.14
Apr 7, 2026
Merged

[3.14] gh-137586: Open external osascript program with absolute path (GH-137584)#148173
hugovk merged 1 commit intopython:3.14from
miss-islington:backport-a0c57a8-3.14

Conversation

@miss-islington
Copy link
Copy Markdown
Contributor

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

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
Copy link
Copy Markdown

python-cla-bot bot commented Apr 6, 2026

All commit authors signed the Contributor License Agreement.

CLA signed

@hugovk hugovk enabled auto-merge (squash) April 6, 2026 17:13
@hugovk hugovk merged commit 8f59d40 into python:3.14 Apr 7, 2026
52 checks passed
@miss-islington miss-islington deleted the backport-a0c57a8-3.14 branch April 7, 2026 03:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type-security A security issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants