Bug 1055594 - src.rpm with URL containig escape sequence cannot be retrieved
Summary: src.rpm with URL containig escape sequence cannot be retrieved
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Copr
Classification: Community
Component: frontend
Version: unspecified
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Miroslav Suchý
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-01-20 15:24 UTC by Simon Gerhards
Modified: 2014-02-06 08:32 UTC (History)
1 user (show)

Fixed In Version: copr-1.25-1
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-02-06 08:32:05 UTC
Embargoed:


Attachments (Terms of Use)
[PATCH] always unqote pkg urls (1.07 KB, patch)
2014-01-21 14:33 UTC, Igor Gnatenko
no flags Details | Diff

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


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