Bug 1055594

Summary: src.rpm with URL containig escape sequence cannot be retrieved
Product: [Community] Copr Reporter: Simon Gerhards <Simon.Gerhards>
Component: frontendAssignee: Miroslav Suchý <msuchy>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: unspecifiedCC: ignatenko
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: copr-1.25-1 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-02-06 08:32:05 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:
Attachments:
Description Flags
[PATCH] always unqote pkg urls none

Description Simon Gerhards 2014-01-20 15:24:04 UTC
The following URL does not work when trying to build a rpm:
https://dl.dropboxusercontent.com/u/44454747/passwordsafe-0.92%2Bgit20140117-3.fc20.src.rpm

The same URL but with "%2B" manually decoded to "+" works:
https://dl.dropboxusercontent.com/u/44454747/passwordsafe-0.92+git20140117-3.fc20.src.rpm

Comment 1 Igor Gnatenko 2014-01-21 14:33:43 UTC
Created attachment 853257 [details]
[PATCH] always unqote pkg urls

Comment 2 Miroslav Suchý 2014-01-21 16:37:38 UTC
Hmm I was tempting if this should bu mockchaing bug... because 
mockchain https://dl.dropboxusercontent.com/u/44454747/passwordsafe-0.92%2Bgit20140117-3.fc20.src.rpm
will create:
Packages successfully built in this order:
/tmp/tmpe6sfkP/passwordsafe-0.92+git20140117-3.fc20.src.rpm

but yes, fixing it on our side is easiest. Your patch looks good. Thanks for contribution.

Commited as c68e2f6