Skip to content

bpo-37363: Add audit events on startup for the run commands#14524

Merged
zooba merged 13 commits intopython:masterfrom
zooba:bpo-37363
Jul 1, 2019
Merged

bpo-37363: Add audit events on startup for the run commands#14524
zooba merged 13 commits intopython:masterfrom
zooba:bpo-37363

Conversation

@zooba
Copy link
Copy Markdown
Member

@zooba zooba commented Jul 1, 2019 •
edited by bedevere-bot
Loading

@zooba zooba requested a review from tiran July 1, 2019 17:34
@tiran
Copy link
Copy Markdown
Member

tiran commented Jul 1, 2019

Fantastic! This simplifies one of my PoCs for EP a lot.

Are these all entry points or is there another entry point for embedded Python interpreters?

@tiran tiran self-assigned this Jul 1, 2019
@zooba
Copy link
Copy Markdown
Member Author

zooba commented Jul 1, 2019

There are definitely more entry points, but if someone else is embedding it then they have full control and audit hooks won't help :)

tiran
tiran reviewed Jul 1, 2019
View reviewed changes
Doc/library/sys.rst
read, so that you can set this hook there. The :mod:`site` module
:ref:`sets this <rlcompleter-config>`.

.. audit-event:: cpython.run_interactivehook hook sys.__interactivehook__
Copy link
Copy Markdown
Member

@tiran tiran Jul 1, 2019

Choose a reason for hiding this comment

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

I don't understand the audit-event syntax. The event takes one argument (hook), but the audit-event has two arguments. What's going on here?

Copy link
Copy Markdown
Member Author

@zooba zooba Jul 1, 2019

Choose a reason for hiding this comment

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

I need to document the syntax somewhere...

Basically, it's:

.. audit-event:: event-name list,of,comma,separated,args anchor

Where the anchor is used for the backlink from the generated table (which appears to not be generating right now...).

@zooba
Copy link
Copy Markdown
Member Author

zooba commented Jul 1, 2019

I can't repro the Azure Pipelines failures locally (on Windows or Ubuntu), so I'm guessing it's environment related. Making one more guess before I start adding lots of diagnostic info :)

@zooba zooba merged commit e226e83 into python:master Jul 1, 2019
@miss-islington
Copy link
Copy Markdown
Contributor

miss-islington commented Jul 1, 2019

Thanks @zooba for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

@zooba zooba deleted the bpo-37363 branch July 1, 2019 23:03
@bedevere-bot
Copy link
Copy Markdown

bedevere-bot commented Jul 1, 2019

GH-14536 is a backport of this pull request to the 3.8 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 1, 2019
@zooba @miss-islington
bpo-37363: Add audit events on startup for the run commands (pythonGH…
b32e924 
…-14524)

(cherry picked from commit e226e83)

Co-authored-by: Steve Dower <steve.dower@python.org>
miss-islington added a commit that referenced this pull request Jul 1, 2019
@miss-islington @zooba
bpo-37363: Add audit events on startup for the run commands (GH-14524)
746992c 
(cherry picked from commit e226e83)

Co-authored-by: Steve Dower <steve.dower@python.org>
lisroach pushed a commit to lisroach/cpython that referenced this pull request Sep 10, 2019
@zooba @lisroach
bpo-37363: Add audit events on startup for the run commands (pythonGH…
51db1ad 
…-14524)
DinoV pushed a commit to DinoV/cpython that referenced this pull request Jan 14, 2020
@zooba @DinoV
bpo-37363: Add audit events on startup for the run commands (pythonGH…
3ed0388 
…-14524)
@zooba zooba mentioned this pull request Nov 27, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tiran tiran tiran left review comments

Assignees

@tiran tiran

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@zooba @tiran @miss-islington @bedevere-bot @the-knights-who-say-ni