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

Incomplete string escaping or encoding #4727

Open
Anu-MT opened this issue Oct 15, 2024 · 2 comments
Open

Incomplete string escaping or encoding #4727

Anu-MT opened this issue Oct 15, 2024 · 2 comments
Assignees
Labels
type:bug Issues that describe a user-facing bug in the project.

Comments

@Anu-MT
Copy link

Anu-MT commented Oct 15, 2024

The regex in the file webapps/frontend/ui/cockpit/tests/pages/process-instance/instance-view.js gives a code ql warning: Incomplete string escaping or encoding.
This replaces only the first occurrence of '>' and '<'.
Fix is to add the global replacement in the regex

@Anu-MT Anu-MT added the type:bug Issues that describe a user-facing bug in the project. label Oct 15, 2024
@Anu-MT
Copy link
Author

Anu-MT commented Oct 16, 2024

I would like to work on it.

@psavidis psavidis self-assigned this Oct 18, 2024
@psavidis
Copy link
Contributor

psavidis commented Oct 21, 2024

Hello @Anu-MT,

Thank you in advance for your interest to provide a contribution :)

As a first step, could you please follow the contribution guidelines to create an issue that follows the structure we have for other issues as well ?

Then we can move on to qualifying this issue.

Best,
Petros

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug Issues that describe a user-facing bug in the project.
Projects
None yet
Development

No branches or pull requests

2 participants