-
Notifications
You must be signed in to change notification settings - Fork 203
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 special characters in filters of HTML tables #806
Comments
Having |
benchexec-2024-03-05_03.06.19.mp4@PhilippWendler |
Sure, would be great! Just make sure to coordinate with #1006 if you need to change related parts of the code (not sure). For |
Error occurs when Also why are we using url to store the filter and other values ? Is there any specific reason ? I think using localstorage or sessionstorage will also satisfy the case and will be much easier to access. |
Yes, URI encoding seems to be the intuitive choice. Using the URL as state storage is by design, because we want to make it possible for people to create links that directly open a specific view of the table. This is extremely useful for many users of BenchExec. |
Not all special characters are handled correctly if a user enters them as a filter for task ids or textual columns because of missing escaping. Some of them just do not work, some actually break the parsing of the URL.
Some characters are actually handled differently depending on whether they are entered on the table tab or in the filter overlay, but not sure whether either of them is working.
We should check and handle at least the following characters, and also add tests for them:
?
#
&
=
(
)
,
*
The text was updated successfully, but these errors were encountered: