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

Špatné zpracovani parametru pro limit načtených položek #630

Closed
oroztocil opened this issue Feb 27, 2024 · 0 comments · Fixed by #631
Closed

Špatné zpracovani parametru pro limit načtených položek #630

oroztocil opened this issue Feb 27, 2024 · 0 comments · Fixed by #631
Assignees
@oroztocil
Labels
čas:krátký časová náročnost: max 8 hodin (1 md) part:backend Týká se API nebo Worker, tj. zpracování, import nebo export dat typ:chyba Chyba, kterou je potřeba opravit ve stávající implementaci

Comments

@oroztocil
Copy link
Collaborator

Kvůli špatné sanitizaci parametru limit na několik API endpointech dochází ke generování nevalidních DB dotazu.

V produkčním server logu je na to spoustu chyb.
@oroztocil oroztocil added typ:chyba Chyba, kterou je potřeba opravit ve stávající implementaci part:backend Týká se API nebo Worker, tj. zpracování, import nebo export dat čas:krátký časová náročnost: max 8 hodin (1 md) labels Feb 27, 2024
@oroztocil oroztocil self-assigned this Feb 27, 2024
oroztocil added a commit that referenced this issue Feb 27, 2024
@oroztocil
#630 Handle limit query param properly
5f48ce8
@oroztocil oroztocil mentioned this issue Feb 27, 2024
Merged
oroztocil added a commit that referenced this issue Mar 5, 2024
@oroztocil
Bugfix/630 wrong limit param handling (#631)
14ff6dc 
* #630 Handle limit query param properly

* Fix eslint 'fix' npm command
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@oroztocil oroztocil

Labels
čas:krátký časová náročnost: max 8 hodin (1 md) part:backend Týká se API nebo Worker, tj. zpracování, import nebo export dat typ:chyba Chyba, kterou je potřeba opravit ve stávající implementaci
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Bugfix/630 wrong limit param handling cityvizor/cityvizor
1 participant
@oroztocil