0016755: 2 [Core] Download struikelt over sommige filenamen

[admin]

---

0016755: 2 [Core] Download struikelt over sommige filenamen
 http://mantis.kennisnet.nl/view.php?id=16755

Als een filenaam een aanhalingsteken bevat zoals het voorbeeld, dan lukt het niet om de file te downloaden op het aangegeven pad.

Gezien op VP-Core acceptatie, EGA-id 5 (SURFmedia testomgeving)
assetid: ZkbRo?5kEY2HR18bEtxPogtPg
mediafileid: JyaZI01k2NkaV1Fd3QPdkgzu

[admin]

<?xml version="1.0"?>
<response>

<header>

<item_count>1</item_count>
<request_process_time>0.3688</request_process_time>
<request_query_count>40</request_query_count>
<request_result>success</request_result>
<request_result_description></request_result_description>
<request_result_id>601</request_result_id>
<request_uri>[GET] /asset/ZkbRo5kEY2HR18bEtxPogtPg/play?mediafile_id=JyaZI01k2NkaV1Fd3QPdkgzu&response=download&user_id=michiel</request_uri>
<vpx_version>1.4.3</vpx_version>

</header>
<items>

<item id="1">

<output> http://download.acceptatie.vpx.kennisnet.nl/WrMh3Dd2JjmHKjkGajpxzLs3/I [^{] Can't Get The Printer To Work.wmv</output>
<content_type></content_type>}

</item>

</items>

</response>

[MC-arjen]

gefixed in 1.6

[Frans]

Does not work, see
 http://www.20.test.surfmedia.nl/medialibrary/item.html?id=9z4Wq2avUgJRePVYAJlmG9Oc

[Frans]

due in 1.6.1

[MC-arjen]

now on acceptatie.

[Michiel.Schok]

We are sorry.
When we navigate to the chosen link on SURFmedia - test where the offending video can be found (  http://www.20.test.surfmedia.nl/medialibrary/item.html?id=9z4Wq2avUgJRePVYAJlmG9Oc ), there is a download button at the first media file.

That button invokes the restcall

/asset/9z4Wq2avUgJRePVYAJlmG9Oc/play?mediafile_id=MJLg3fysoUu3PTpNWUHgEzI4&response=download&user_id=michiel

which returns

    <item id="1">
      <output>http://download.acceptatie.vpx.kennisnet.nl/TcNQ4TzEESoDIrlzF4OYLnHW/I Can't Get The Printer To Work.wmv</output>
      <content_type></content_type>
    </item>

And when issued in a browser, my browser reports 'File not found' with a HTML status code of 200-OK, which is given to my browser by 'download.acceptatie.vpx.kennisnet.nl'

Still not OK, not accepted.

[MC-arjen]

De oplossing was hier om de bestandsname te urlencoderen, bovenstaande filename zal nu een %27 geven voor een quote, het is echter wel downloadable, is dit acceptabel?

[MC-arjen]

The solution for this was to urlencode the file, quotes are changed in %27.

[Michiel.Schok]

Satisfactory.

[Michiel.Schok]

Regression tests show that it is broken in VP-core 1.7.1
Issue is reopened, and moved to project MediaMosa 1.7

[forgacs]

Ready. It is on the SVN.

[peter]

controleren en meenemen in 2.0

[robert]

Tested in 2.1; Unit test creates a file called "plus'q uote" and tries to download it, this is done successful in 2.1 sandbox test.

Moved to 2.1 for final test for approval on acceptation release.

[Michiel.Schok]

I think Jens will appreciate that his files are now downloadable.