FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-138577: Mention limitations of `getpass.getpass(echo_char=...)` by yagggi · Pull Request #138677 · python/cpython · GitHub

/ cpython Public

gh-138577: Mention limitations of getpass.getpass(echo_char=...) - #138677

Merged
picnixz merged 5 commits into
python:mainfrom
yagggi:getpass-doc
Sep 9, 2025
Merged

gh-138577: Mention limitations of getpass.getpass(echo_char=...)#138677
picnixz merged 5 commits into
python:mainfrom
yagggi:getpass-doc

Conversation

yagggi commented Sep 8, 2025
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Contributor

python-cla-bot Bot commented Sep 8, 2025
edited
Loading

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.

picnixz changed the title gh-138577: Mention unexpected behavior of getpass.getpass() in document gh-138577: Mention limitations of getpass.getpass(echo_char=...) Sep 8, 2025
Comment thread Doc/library/getpass.rst Outdated
Comment thread Doc/library/getpass.rst Outdated
Comment thread Doc/library/getpass.rst Outdated
yagggi and others added 2 commits September 9, 2025 02:11
Co-authored-by: Brian Schubert <brianm.schubert@gmail.com>
Comment thread Doc/library/getpass.rst Outdated
Comment thread Doc/library/getpass.rst Outdated
picnixz added the needs backport to 3.14 bugs and security fixes label Sep 8, 2025
picnixz merged commit 074f3b2 into python:main Sep 9, 2025
29 checks passed
github-project-automation Bot moved this from Todo to Done in Docs PRs Sep 9, 2025

Copy link
Copy Markdown

Thanks @yagggi for the PR, and @picnixz for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

picnixz commented Sep 9, 2025

Copy link
Copy Markdown
Member

Thank you for the fix!

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 9, 2025
…s(echo_char=...)` (pythonGH-138677)

In bf8bbe9, `getpass.getpass` gained
the ability to provide keyboard feedback through `echo_char`.

On Unix, line editing shortcuts such as Ctrl+U were previously handled
as the terminal operates in canonical mode (see termios(3)). However,
since keyboard feedback requires to switch to noncanonical mode, this
now results in an inconsistency when `getpass.getpass` uses `echo_char`
as those shortcuts are no more supported. This limitation is specific
to Unix and does not affect Windows users where line editing shortcuts
were never supported.
(cherry picked from commit 074f3b2)

Co-authored-by: yagggi <fakepoet0101@gmail.com>

bedevere-app Bot commented Sep 9, 2025

Copy link
Copy Markdown

GH-138696 is a backport of this pull request to the 3.14 branch.

bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Sep 9, 2025
hugovk pushed a commit that referenced this pull request Sep 9, 2025
…ss(echo_char=...)` (GH-138677) (#138696)

Co-authored-by: yagggi <fakepoet0101@gmail.com>

yagggi commented Sep 9, 2025

Copy link
Copy Markdown
Contributor Author

Thanks @picnixz for your patience and guidance, glad to do something to CPython.

lkollar pushed a commit to lkollar/cpython that referenced this pull request Sep 9, 2025
…s(echo_char=...)` (python#138677)

In bf8bbe9, `getpass.getpass` gained
the ability to provide keyboard feedback through `echo_char`.

On Unix, line editing shortcuts such as Ctrl+U were previously handled
as the terminal operates in canonical mode (see termios(3)). However,
since keyboard feedback requires to switch to noncanonical mode, this
now results in an inconsistency when `getpass.getpass` uses `echo_char`
as those shortcuts are no more supported. This limitation is specific
to Unix and does not affect Windows users where line editing shortcuts
were never supported.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL