Skip to content
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

Disable allow_abbrev from Python scripts using argparse #29

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

juhaylinen
Copy link

Python's argparse library, by default, allows shortening of command line arguments. This can introduce silent failures when shortened commands are used and another command is added to the script which uses that name.

This change was requested on Mbed-TLS/mbedtls#9294

Copy link
Contributor

@gilles-peskine-arm gilles-peskine-arm left a comment

Choose a reason for hiding this comment

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

LGTM

@juhaylinen
Copy link
Author

Rebased

Python's argparse library, by default, allows shortening of
command line arguments. This can introduce silent failures when
shortened commands are used and another command is added to the
script which uses that name.

Signed-off-by: Juha Ylinen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants