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

["David Jencks (JIRA)" <ji...@apache.org>]

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

David Jencks updated FELIX-2195:
--------------------------------

    Attachment: FELIX-2195.diff

Use harmony code, not the wrong approach.

> 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
>         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.