The Wayback Machine - https://web.archive.org/web/20200530185537/https://github.com/pyrogram/pyrogram/issues/328
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

Plain 2FA password input #328

Open
RadRussianRus opened this issue Oct 21, 2019 · 3 comments
Open

Plain 2FA password input #328

RadRussianRus opened this issue Oct 21, 2019 · 3 comments

Comments

@RadRussianRus
Copy link

@RadRussianRus RadRussianRus commented Oct 21, 2019

When entering 2FA password it is shown in plain text. This is because it uses input:

if not self.password:
self.password = input("Enter password (empty to recover): ")

I suggest to replacing it with with getpass.getpass().

@zeroone2numeral2
Copy link
Contributor

@zeroone2numeral2 zeroone2numeral2 commented Oct 22, 2019

iirc getpass was causing some login issues when using pyrogram from Windows or some specific terminals, see this and this discussions in the chat

@delivrance
Copy link
Member

@delivrance delivrance commented Oct 22, 2019

Also, I'm not aware of any way of making getpass async.

@RadRussianRus
Copy link
Author

@RadRussianRus RadRussianRus commented Oct 23, 2019

It uses on login, and manual login process (where this is used) is not async anyway. Or I'm missing something?

Though problems on other terminals is more serious. I'd suggest to use escape codes, but I think problems will be the same.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.
Morty Proxy This is a proxified and sanitized view of the page, visit original site.