Bug 1263921

Summary: Validate group names during edit to not contain '/'
Product: [Retired] Beaker Reporter: matt jia <mjia>
Component: web UIAssignee: Blake McIvor <bmcivor>
Status: CLOSED CURRENTRELEASE QA Contact: tools-bugs <tools-bugs>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: developCC: bmcivor, dcallagh, dowang, mjia, rjoost
Target Milestone: 24.0Keywords: Patch, Triaged
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-02-21 18:48:46 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description matt jia 2015-09-17 05:51:38 UTC
Description of problem:

In https://bugzilla.redhat.com/show_bug.cgi?id=1251356, the group page is ported to Backbone. It supports to have a group name containing slash '\' and the group url path will contain %2f which is an escaped '/' which will cause 404 error. This is because of a limitation in WSGI(https://github.com/mitsuhiko/flask/issues/900). 


Version-Release number of selected component (if applicable):


How reproducible:

Always

Steps to Reproduce:
1. Update a group name with '/'
2. Go to this group page

Actual results:

404 Not Found 

Expected results:

The page can be loaded successfully.

Comment 1 Blake McIvor 2016-08-03 04:04:29 UTC
https://gerrit.beaker-project.org/#/c/5109/

Comment 4 Roman Joost 2016-08-16 04:20:13 UTC
Dear Wang Dong,

Is it possible you have tested this with the wrong assumption? This bug is about group names shouldn't contain a slash '/'. You've tested this using a backslash '\'. Could you test it again please?

Comment 7 Dan Callaghan 2017-02-21 18:48:46 UTC
Beaker 24.0 has been released.