Bug 1263921 - Validate group names during edit to not contain '/'
Summary: Validate group names during edit to not contain '/'
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Beaker
Classification: Retired
Component: web UI
Version: develop
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: 24.0
Assignee: Blake McIvor
QA Contact: tools-bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-09-17 05:51 UTC by matt jia
Modified: 2017-02-21 18:48 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-02-21 18:48:46 UTC
Embargoed:


Attachments (Terms of Use)

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.


Note You need to log in before you can comment on or make changes to this bug.