incubator-chemistry-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hinse ter Schuur (JIRA)" <>
Subject [jira] Commented: (CMIS-111) Content streams not correctly served
Date Wed, 03 Feb 2010 14:11:28 GMT


Hinse ter Schuur commented on CMIS-111:

I've tried to reproduce with a fresh checkout of Chemistry and couldn't reproduce the problem.
The difference with my normal situation is that I changed the dependency on Abdera. My local
copy of Chemistry depends on Abdera 1.0 instead of 0.4.0-incubating.

Abdera 1.0 uses this isBinary() method to determine whether it should use either getOutputStream
or getWriter on the ServletResponse.
if (!request.getMethod().equals("HEAD") && context.hasEntity()) {
    if (context.isBinary()) {
    } else {

So it's not a real problem for the current version, but it will become a problem when Chemistry
starts to depend on Abdera 1.0.

> Content streams not correctly served
> ------------------------------------
>                 Key: CMIS-111
>                 URL:
>             Project: Chemistry
>          Issue Type: Bug
>          Components: server
>            Reporter: Hinse ter Schuur
>         Attachments: media-binary.txt
> When downloading (binary) content from the server the contents are not correct. This
seems to be because a SizedMediaResponseContext is  treated as non-binary, resulting in written
out to the request context´ s writer instead of outputstream.
> A simple reproduction case is: run the org.apache.chemistry.test.MainServlet from the
chemistry-test module upload a zip file and try to download it again. The resulting download
does not have the same size as the original and the archive is corrupted.
> A simple fix will be to make the SizedMediaResponseContext binary. I'll attach a patch
file for this.

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

View raw message