[weld-issues] [JBoss JIRA] (WELD-954) URIs escaped twice in URLScanner

[Emond Papegaaij (Commented) (JIRA)]

    [ https://issues.jboss.org/browse/WELD-954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12638283#comment-12638283 ] 

Emond Papegaaij commented on WELD-954:
--------------------------------------

This will probably work, but why is all this escaping needed anyway? Isn't it possible to do the entire scanning directly on the URLs, using url.openStream()? Your 'fix' will work for us, but will break with other characters. What if someone uses a directory called '%20 %25'?
                
> URIs escaped twice in URLScanner
> --------------------------------
>
>                 Key: WELD-954
>                 URL: https://issues.jboss.org/browse/WELD-954
>             Project: Weld
>          Issue Type: Bug
>          Components: Bootstrap and Metamodel API
>    Affects Versions: 1.1.2.Final
>            Reporter: Emond Papegaaij
>            Assignee: Ales Justin
>
> The fix for WELD-834 can cause spaces to be escaped twice in an URL. This leads to errors like: Error handling file /var/lib/jenkins/jobs/Wicket%2520Weld/workspace/wicket-cdi-examples/target/classes
> Notice how 'Wicket%20Weld' is escaped to 'Wicket%2520Weld'. https://github.com/papegaaij/wicket-cdi demonstrates the problem, clone the project in a directory with a space in the name, and run 'mvn install -Dwicket.version=1.5-RC5.1'.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira