Bug 1380820

Summary:	accounts containining slash breaks webui functionality
Product:	[Internal] Red Hat Internal Copr	Reporter:	Tomas Dolezal <todoleza>
Component:	backend	Assignee:	Pavel Raiskup <praiskup>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	unspecified	CC:	lpol
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-12-14 07:29:22 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:

Description Tomas Dolezal 2016-09-30 16:39:10 UTC

Description of problem:
COPR's webui cannot cope with accounts like 'type/host.tld', resulting in links leading to 404 error not found. This happens on account page or when creating new repo.

Version-Release number of selected component (if applicable):
Build 1.78-10.fc23

How reproducible:
always

Steps to Reproduce:
log in with account containing slash (via kerberos for example)

Actual results:
successful login, features broken

Comment 2 Tomas Dolezal 2016-10-04 17:05:59 UTC

with latest fix where the slash '/' was substituted with underscore '_' it's no possible work in the UI. repofiles are generated as well with proper urls. repo provides installable sources.

gpgkey file is unavailable in this and other repos but gpgcheck is disabled anyway.

Comment 3 Pavel Raiskup 2016-10-04 17:39:04 UTC

(In reply to Tomas Dolezal from comment #2)
> with latest fix where the slash '/' was substituted with underscore '_' it's
> no possible work in the UI.

Thanks for testing, can you be more specific here?  Probably typo 's/no/not/'
or 's/no/now'. :)

Comment 4 Tomas Dolezal 2016-10-04 17:42:48 UTC

(In reply to Pavel Raiskup from comment #3)
> (In reply to Tomas Dolezal from comment #2)
> > with latest fix where the slash '/' was substituted with underscore '_' it's
> > no possible work in the UI.
> 
> Thanks for testing, can you be more specific here?  Probably typo 's/no/not/'
> or 's/no/now'. :)
This is a funny typo. I meant 'now', it works :) sorry for confusion

Comment 5 Pavel Raiskup 2016-10-05 05:57:02 UTC

Upstream fix:
https://github.com/fedora-copr/copr/commit/a999c6a2f72f5b833d67b82b