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

Hide password with another flag for questionary.password #321

Open
Crissal1995 opened this issue Sep 14, 2023 · 3 comments · May be fixed by #421
Open

Hide password with another flag for questionary.password #321

Crissal1995 opened this issue Sep 14, 2023 · 3 comments · May be fixed by #421
Labels
Enhancement New feature or request

Comments

@Crissal1995
Copy link

Describe the problem

Right now, questionary.password().ask() will replace every typed character with an asterisk.

Would be nice if the output could be completely removed, in order to ensure password length is not leaked when looking at the screen/logs of the user.

Describe the solution

The solution should be that no character is displayed when typing a password, similar to how getpass reads the password from the prompt.

Alternatives considered

No response

@Celina1801
Copy link

Celina1801 commented Oct 17, 2024

Hello @tmbo ,
I would like to work on this issue. Could you please assign it to me?
Thank you!

@kiancross
Copy link
Collaborator

Hi @Celina1801. Thank you for offering to work on this issue. As far as I am aware, nobody else currently is. Please submit a PR if you get a working version.

@Crissal1995
Copy link
Author

I've made a PR in the dependency repository that will solve this issue: prompt-toolkit/python-prompt-toolkit#1962
What will be left is just provide a new option to password to hide the password prompt on-demand.

@Celina1801 Celina1801 linked a pull request Jan 21, 2025 that will close this issue
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants