-
Notifications
You must be signed in to change notification settings - Fork 234
[Meta] PSES should handle unusual path characters properly #714
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
Comments
May also be worth checking with the more unusual UTF8 characters e.g. Runes is a common one we've had trouble with in ruby (puppet) land and Windows.
|
Not denying that that's a good test case, but were people using runic characters in Ruby/puppet?? PowerShell (Core) and VSCode theoretically handle UTF-8 properly. Our problems with encodings come from (1) Windows PowerShell defaulting to ISO-8859-1/latin-1/CP1252, (2) C# using UTF-16 and (3) the LSP encoding in UTF-8 but specifying offsets in UTF-16. However, we've been bitten less by encoding issues and much more by:
I think @rkeithhill has sorted most of these last issues out in a recent PR, but it's an ongoing bug hunt. But yeah, a rune string we could easily add to the tests. |
But there's more work to be done to clean up ClientFilePaths that get sent back to the client from PSES> |
Oh hell no! But it's on the extreme end of UTF8 usage. If it works with runes, then it'll probably work everywhere else. |
There are a few open issues that look like PSES is crashing when it comes across a strange character.
#
: Crash "Errors caught while waiting for EditorServicesHost to complete execution" between two conflicting file vscode-powershell#1453.
: Debug adapter process has terminated unexpectedly vscode-powershell#918 (comment)The text was updated successfully, but these errors were encountered: