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

Please include build with utf-8 included #2266

Open
heitbaum opened this issue Aug 27, 2024 · 3 comments
Open

Please include build with utf-8 included #2266

heitbaum opened this issue Aug 27, 2024 · 3 comments

Comments

@heitbaum
Copy link

Summary of the new feature / enhancement

With QR functionality MS Entra ID ubuntu/authd#497 being introduced in ubuntu auth/sshd/ - please enable utf-8. I have tested with the latest https://github.com/PowerShell/Win32-OpenSSH/releases/tag/v9.5.0.0p1-Beta

ref:

Proposed technical implementation details (optional)

No response

@tgauth tgauth transferred this issue from PowerShell/openssh-portable Aug 27, 2024
@mgkuhn
Copy link

mgkuhn commented Nov 6, 2024

Display of UTF-8 characters has worked for a long time in normal SSH sessions when you invoke ssh in Windows Terminal to connect to a Linux machine, including on current Windows 11 with OpenSSH_for_Windows_9.5p1.

UTF-8 example files such as

$ curl https://www.cl.cam.ac.uk/~mgk25/ucs/examples/UTF-8-demo.txt

display very nicely via OpenSSH for WIndows in Windows Terminal.

@mgkuhn
Copy link

mgkuhn commented Nov 6, 2024

Is the problem perhaps merely that during KbdInteractiveAuthentication that non-ASCII bytes in challenge strings are not passed through transparently, but are escaped as octal numbers with backslashes?

@mgkuhn
Copy link

mgkuhn commented Nov 6, 2024

This seems to be an openssh-portable upstream issue, with an open PR at openssh/openssh-portable#452 and therefore may not actually be specific to OpenSSH for Windows.

It looks as if such QR codes rendered using UTF-8 block-graphic characters in PAM-based kbd-interactive dialogues only work on Ubuntu's OpenSSH fork so far, because Ubuntu had patched their own version of OpenSSH, as upstream OpenSSH so far escapes many bytes in these PAM dialogue strings via strnvis().

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants