Actions
action #95030
open[easy][beginner] Better, complete validation of parameters on /tests/overview
Start date:
2021-06-29
Due date:
% Done:
0%
Estimated time:
Tags:
Description
Motivation¶
The tests overview page doesn't validate its input. Omitting the group leads to the site picking up some results from the entire db. Invalid values lead to server errors.
Examples:
- https://openqa.opensuse.org/tests/overview
- https://openqa.opensuse.org/tests/overview?groupid=spam
- https://openqa.opensuse.org/tests/overview?groupid=-5
Acceptance criteria¶
- AC1: Document use case w/o group
- AC2: Request fails if invalid groups are specified
Suggestion¶
- Add message to web UI e.g. "showing failed results from all groups" or something
- Add parameter validation to lib/OpenQA/WebAPI/Controller/Test.pm
Further details¶
entrance level issue
Updated by okurz over 3 years ago
- Copied from action #94901: Simple validation of parameters on /tests/overview added
Updated by okurz over 2 years ago
- Tags set to easy, beginner, entrance level
- Subject changed from Better, complete validation of parameters on /tests/overview to [easy][beginner] Better, complete validation of parameters on /tests/overview
- Description updated (diff)
Actions