action #104631
closedcoordination #154768: [saga][epic][ux] State-of-art user experience for openQA
coordination #154771: [epic] Improved test developer user experience
[easy][beginner] WebUI: No error message when putting invalid characters in machines/testsuites settings size:S
0%
Description
Observation¶
When I go to "Machines", click on the edit button and put presumable invalid characters into a settings name, e.g. HDDSIZEGB!!=20
and click the Save button, I don't get an error message.
Apparently an API request is made http://127.0.0.1:9526/api/v1/machines/1001
which returns 200 OK and {"result":1}
. But the new setting is not saved, just silently ignored.
If I add a new setting with invalid characters, e.g. foo!!=bar
, then it is saved as foo=bar
.
Acceptance criteria¶
- AC1: Invalid settings keys like
!
are explicitly not allowed on admin tables and not silently removed (or at least there is a user facing warning about it).
Suggestions¶
- Be sure to rebase on top of recently merged https://github.com/os-autoinst/openQA/pull/5973
- Lookup in neighboring code if there is a range of acceptable characters (at least [[:alnum:]_]) or a blocklist containing at least
!
- Consider to also add tests on the lower level, e.g. API route in t/api/05-machines.t
- Extend the according test in t/ui/13-admin.t to expect clear feedback
- Implement accordingly
- Refactor/simplify the code as necessary
Further details¶
entrance level issue
Files
Updated by okurz over 2 years ago
- Tags set to ui, ux, easy, beginner, entrance level
- Subject changed from WebUI: No error message when putting invalid characters in machines/testsuites settings to [easy][beginner] WebUI: No error message when putting invalid characters in machines/testsuites settings
- Description updated (diff)
Updated by mkittler 27 days ago ยท Edited
The behavior is still reproducible. I'd block this on #64412 / https://github.com/os-autoinst/openQA/pull/5973. After that we'll see how the behavior has changed. I just tested it with the PR applied and the behavior is still reproducible. To avoid too many changes at once I still suggest to wait until the PR has been merged.
Updated by okurz 24 days ago
- Subject changed from [easy][beginner] WebUI: No error message when putting invalid characters in machines/testsuites settings to [easy][beginner] WebUI: No error message when putting invalid characters in machines/testsuites settings size:S
- Description updated (diff)
- Status changed from New to Workable
Updated by openqa_review 23 days ago
- Due date set to 2024-10-23
Setting due date based on mean cycle time of SUSE QE Tools