-
-
Notifications
You must be signed in to change notification settings - Fork 88
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
Handle windows carriage returns (CRLF) in Filter form #127
Comments
I've just hit this issue in Chrome. The worst part about it is that if you don't know about it, you have absolutely no clue what's happening and what's wrong with the perfectly valid-looking And I don't think it occurs that rarely - yes, it doesn't happen when you write the You can check the presence of CRLF endings for example by pasting the clipboard contents to a hex editor like HxD. If you see
I can confirm this doesn't happen in Firefox - apparently Firefox sanitizes the input you copy in the "Filter" field by replacing the CR LF endings to just LF. Chrome doesn't do this and sends the
You have the ASCII codes the other way around - you actually meant 13 + 10 (13 is CR and 10 is LF; it can be written as |
Do you have an example to reproduce the issue? It would be helpful to see a specific case where this occurs. Could you also check if the problem persists with the latest rewrite, which now uses the Monaco Editor? Let us know if you find anything! |
Hello,
Whenever I copy multiple lines from my IDE into the jqplay services url, I would get this error
unexpected INVALID_CHARACTERm expecting $end
I did a bit of debugging locally and discovered that when I copy from my clipboard on Windows 10 into a Microsoft Edge browser (version 105.0.1343.50) that the line endings are ascii characters 10 + 13, as opposed to just 13.
If I remove the line endings and then recreate them inside the Filter form, everything is fine
This doesn't happen when I do the same thing in Firefox so seems to be OS/browser specific.
I did try stripping the CRLF line endings in the GO source code but don't know enough about the GO language to know exactly where this snippet needs to be put.
The text was updated successfully, but these errors were encountered: