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 bracketed paste #177

Merged
merged 1 commit into from
Mar 13, 2024
Merged

Conversation

pkern
Copy link
Member

@pkern pkern commented Mar 13, 2024

In some settings we seem to end up with users' terminals being confused and willing to send bracketed paste escape codes, which then cause authentication to fail. Even worse it looks like the special characters are not displayed in some shells, thus confusing users as to what just happened.

Although to be honest: I have not found the specific environment where this breaks in a reproducible fashion and this is effectively defence in depth.

In some settings we seem to end up with users' terminals being confused
and willing to send bracketed paste escape codes, which then cause
authentication to fail. Even worse it looks like the special characters
are not displayed in some shells, thus confusing users as to what just
happened.

Although to be honest: I have not found the specific environment where
this breaks in a reproducible fashion and this is effectively defence
in depth.
Copy link
Collaborator

@lukegb lukegb left a comment

Choose a reason for hiding this comment

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

¯\_(ツ)_/¯

@pkern pkern merged commit a00cf30 into google:master Mar 13, 2024
7 checks passed
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