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.

Sign up for GitHub

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

Jump to bottom

Type all url params for state management #394

Closed
jacovinus opened this issue Jan 17, 2024 · 1 comment
Closed

Type all url params for state management #394

jacovinus opened this issue Jan 17, 2024 · 1 comment
Assignees
@jacovinus
Labels
bug Something isn't working

Comments

@jacovinus
Copy link
Collaborator

Describe the bug.

We have a security issue pointing to the parsing of url params that are input mannually.
For solving this we will type all acceptable url params and filter the ones that are not allowed

Reproduction

Type custom url params
the application should filter the ones that are not part of the application

Application version

0.23.0

Browser version

No response

Logs

No response

Anything else?

No response
@jacovinus jacovinus added the bug Something isn't working label Jan 17, 2024
@jacovinus jacovinus self-assigned this Jan 17, 2024
jacovinus added a commit that referenced this issue Jan 17, 2024
@jacovinus
Merge pull request #395 from metrico/fix/type_url_params
290c553 
Fix:#394 type url params
@jacovinus
Copy link
Collaborator Author

fixed at #395

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jacovinus jacovinus

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jacovinus