[
https://issues.apache.org/jira/browse/SHINDIG-1136?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12738295#action_12738295
]
Jinhui Du commented on SHINDIG-1136:
------------------------------------
Thanks for your comments.
- In the RPC method the Content-Type is multipart/form-data and both
php://input.and $HTTP_RAW_POST_DATA are not available. The json converter will
work as the default converter. In the REST method if the Content-Type is set to
multipart/form-data it works. I'm working on the case that the Content-Type is
image/* audio/* or video/*.
- The container should take the responsibility to verify the uploaded files.
In Partuza reference implementation the uploaded file is firstly moved to the
"albums" folder via the move_uploaded_file method as a temporary file then is
read using the image library. Finally the content read by the image library is
written into the albums folder and the temporary file is deleted. As there are
only image libraries in the current code, only image is support now in partuza.
- As php handles the multipart/form logic the container can easily access the
uploaded files via the $_FILES global variable. We may not need to pass the
content data to the container.
I'm working on the the content-type image/*, video/*, audio/* and making the
createActivity in partuza supports content upload. I'll send out the CL later.
> Content upload for php shindig
> ------------------------------
>
> Key: SHINDIG-1136
> URL: https://issues.apache.org/jira/browse/SHINDIG-1136
> Project: Shindig
> Issue Type: New Feature
> Components: PHP
> Environment: php 5.2.4
> Reporter: Jinhui Du
>
> The URL for code review is http://codereview.appspot.com/98103/show
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
