Skip to content

[3.12] gh-118633: Add warning regarding the unsafe usage of eval and exec (GH-118437) #126162

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 30, 2024

Conversation

brianschubert
Copy link
Member

@brianschubert brianschubert commented Oct 30, 2024

  • Add warning regarding the unsafe usage of eval

  • Add warning regarding the unsafe usage of exec

  • Move warning under parameters table

  • Use suggested shorter text

Co-authored-by: Jelle Zijlstra [email protected]

  • Use suggested shorter text

Co-authored-by: Jelle Zijlstra [email protected]

  • Improve wording as suggested

(cherry picked from commit 00e5ec0)

Co-authored-by: Daniel Ruf [email protected]
Co-authored-by: Kirill Podoprigora [email protected]
Co-authored-by: Jelle Zijlstra [email protected]


📚 Documentation preview 📚: https://cpython-previews--126162.org.readthedocs.build/

…l and exec (pythonGH-118437)

* Add warning regarding the unsafe usage of eval

* Add warning regarding the unsafe usage of exec

* Move warning under parameters table

* Use suggested shorter text

Co-authored-by: Jelle Zijlstra <[email protected]>

* Use suggested shorter text

Co-authored-by: Jelle Zijlstra <[email protected]>

* Improve wording as suggested

---------

(cherry picked from commit 00e5ec0)

Co-authored-by: Daniel Ruf <[email protected]>
Co-authored-by: Kirill Podoprigora <[email protected]>
Co-authored-by: Jelle Zijlstra <[email protected]>
@bedevere-app bedevere-app bot added docs Documentation in the Doc dir skip news labels Oct 30, 2024
@hauntsaninja hauntsaninja merged commit b69bb1e into python:3.12 Oct 30, 2024
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation in the Doc dir skip news type-security A security issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants