Skip to content

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

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

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

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

@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
@hugovk hugovk merged commit e70ff77 into python:3.10 Apr 7, 2026
15 checks passed
@miss-islington miss-islington deleted the backport-a0c57a8-3.10 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