You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Martin Grigorov (JIRA)" <ji...@apache.org> on 2013/05/02 17:56:17 UTC
[jira] [Resolved] (WICKET-4934) DownloadLink uses wrong encoding
for spaces/non-ASCII characters
[ https://issues.apache.org/jira/browse/WICKET-4934?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Martin Grigorov resolved WICKET-4934.
-------------------------------------
Resolution: Fixed
Sven's patch is applied.
> DownloadLink uses wrong encoding for spaces/non-ASCII characters
> ----------------------------------------------------------------
>
> Key: WICKET-4934
> URL: https://issues.apache.org/jira/browse/WICKET-4934
> Project: Wicket
> Issue Type: Improvement
> Components: wicket
> Affects Versions: 6.4.0
> Environment: IE7/8, FF 17.0.1, Chrome 23
> Reporter: Matthias Piepkorn
> Assignee: Martin Grigorov
> Priority: Minor
> Labels: download
> Fix For: 7.0
>
> Attachments: DownloadLink.patch, WICKET-4934.patch
>
>
> When downloading a file with spaces and non-ASCII characters in the filename using DownloadLink, all browsers I tested show a wrong filename in the file download dialog.
> For a file named Test 'ä'.txt, I got the following output from different browsers:
> IE: Test+'ä'.txt
> FF: Test+%27%C3%A4%27.txt
> Chrome: Test+%27ä%27.txt
> I have attached a patch that fixes the issue for me. The problem is that
> a) the used URL encoding is wrong. PATH encoding, not QUERY encoding is required
> b) only IE needs the filename to be encoded, other browsers seem to work with the original name
> See e.g. http://stackoverflow.com/questions/93551/how-to-encode-the-filename-parameter-of-content-disposition-header-in-http for a discussion of the underlying issue.
> Note: I do not believe the attached patch is the optimal fix for this issue. I would prefer removing the encoding handling from DownloadLink and putting it into AbstractResource.setResponseHeaders(), so other components using attachments benefit, too.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira