[jira] Resolved: (FELIX-2195) Using URLDecoder.decode on locations is wrong

["Karl Pauls (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/FELIX-2195?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Karl Pauls resolved FELIX-2195.
-------------------------------

       Resolution: Fixed
    Fix Version/s: felix-3.0.0

The patch is trunk (r926330). I did massage it a bit to not depend on java5 and fit the code style. Please close if it works for you.

> Using URLDecoder.decode on locations is wrong
> ---------------------------------------------
>
>                 Key: FELIX-2195
>                 URL: https://issues.apache.org/jira/browse/FELIX-2195
>             Project: Felix
>          Issue Type: Bug
>          Components: Framework
>    Affects Versions: felix-2.0.4
>            Reporter: David Jencks
>            Assignee: Karl Pauls
>             Fix For: felix-3.0.0
>
>         Attachments: FELIX-2195.diff
>
>
> URLDecoder is supposed to be for decoding form goo stuffed into a url, not for decoding the path component of a url.  Using it for this produces wrong results.  In partucular, mac os x likes to make temporary files with strings like "+++++++" in the path.  When run through URLDecoder these turn into spaces which means felix can't find the file you just gave it a valid file system location for.  Since equinox does this correctly we can't use felix in geronimo until this is fixed.
> IIUC URLDecoder is used as an attempt to maintain compatibility with ancient vms that lack URI and its correct decoding behavior.  Fortunately apache harmony implements URI so the attached patch borrows the implementation from harmony to provide correct behavior on antique vms.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.