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

Fix the `sys.remote_exec` row in the audit events table by maurycy · Pull Request #158940 · python/cpython · GitHub

  1. python
  2. cpython

Repository navigation

Fix the sys.remote_exec row in the audit events table - #158940

Merged
pablogsal merged 1 commit into
python:mainfrom
maurycy:remote-exec-audit-event-args
Oct 7, 2026
ConversationCommits (1)ChecksFiles changed
Merged

Fix the sys.remote_exec row in the audit events table#158940
pablogsal merged 1 commit into
python:mainfrom
maurycy:remote-exec-audit-event-args

Conversation

maurycy commented Oct 6, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Basically:

https://docs.python.org/3.16/library/audit_events.html

The sys.remote_exec row shows only pid, while there's script:

cpython/Python/sysmodule.c

Line 2479 in ea0ee92

if (PySys_Audit("sys.remote_exec", "iO", pid, script) < 0) {

and links to sys.html#script_path that doesn't exist.

since 1ddfe59, so likely good to backport to 3.14 and 3.15; so smol that I'm not opening a separate issue.

bedevere-app Bot added awaiting review docs Documentation in the Doc dir skip news labels Oct 6, 2026

maurycy commented Oct 6, 2026

Copy link
Copy Markdown
Contributor Author

cc @pablogsal @vstinner

read-the-docs-community Bot commented Oct 6, 2026

Copy link
Copy Markdown

Documentation build overview

📚 cpython-previews | 🛠️ Build #34980425 | 📁 Comparing 0919a86 against main (1818fba)

  🔍 Preview build  

1 file changed
± library/audit_events.html

vstinner added skip issue needs backport to 3.14 bugs and security fixes needs backport to 3.15 pre-release feature fixes, bugs and security fixes labels Oct 6, 2026
vstinner approved these changes Oct 6, 2026
View reviewed changes

vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM. I'm always confused by spaces vs commas in the audit-event: markup.

pablogsal merged commit 2639fd6 into python:main Oct 7, 2026
47 checks passed

miss-islington-app Bot commented Oct 7, 2026

Copy link
Copy Markdown

Thanks @maurycy for the PR, and @pablogsal for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14, 3.15.
🐍🍒⛏🤖

pablogsal commented Oct 7, 2026

Copy link
Copy Markdown
Member

Thanks a lot!

bedevere-app Bot commented Oct 7, 2026

Copy link
Copy Markdown

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

bedevere-app Bot removed the needs backport to 3.15 pre-release feature fixes, bugs and security fixes label Oct 7, 2026

bedevere-app Bot commented Oct 7, 2026

Copy link
Copy Markdown

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

bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Oct 7, 2026
maurycy deleted the remote-exec-audit-event-args branch October 7, 2026 07:10
hugovk pushed a commit that referenced this pull request Oct 7, 2026
[3.15] Fix the sys.remote_exec row in the audit events table (GH-15…
19fd4df 
…8940) (#158949)

Co-authored-by: Maurycy Pawłowski-Wieroński <maurycy@maurycy.com>
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
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

vstinner vstinner approved these changes

Assignees

No one assigned

Labels

docs Documentation in the Doc dir skip issue skip news

Projects

Docs PRs
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

Back | FazBrowse Home | New Git URL