Bug 1380820 - accounts containining slash breaks webui functionality
Summary: accounts containining slash breaks webui functionality
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Internal Copr
Classification: Internal
Component: backend
Version: unspecified
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ---
Assignee: Pavel Raiskup
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-09-30 16:39 UTC by Tomas Dolezal
Modified: 2016-12-14 07:29 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-12-14 07:29:22 UTC


Attachments (Terms of Use)

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


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