[
https://issues.apache.org/jira/browse/SHINDIG-1555?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13069340#comment-13069340
]
[email protected] commented on SHINDIG-1555:
--------------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/991/#review1163
-----------------------------------------------------------
Looks like some of my previous code review comments weren't addressed. Please
make sure you fix those as well. The refactoring we talked about looks good to
me.
- Ryan
On 2011-07-21 14:59:57, Michael Brockhurst wrote:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/991/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2011-07-21 14:59:57)
bq.
bq.
bq. Review request for Ryan Baxter and Stanton Sievers.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. Implemented the function doPost for the shindig ProxyServlet class.
bq.
bq.
bq. This addresses bug SHINDIG-1555.
bq. https://issues.apache.org/jira/browse/SHINDIG-1555
bq.
bq.
bq. Diffs
bq. -----
bq.
bq.
http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/ProxyHandler.java
1141997
bq.
http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/ProxyServlet.java
1141997
bq.
http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/servlet/ProxyServletTest.java
1141997
bq.
bq. Diff: https://reviews.apache.org/r/991/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq. Created a gadget that made a post request from a proxy server using the
following lines of code:
bq.
bq. var proxyURL = gadgets.io.getProxyUrl("https://www.website.com");
bq. var request = new XMLHttpRequest();
bq. request.open("POST", proxyURL, false);
bq.
bq.
bq. Thanks,
bq.
bq. Michael
bq.
bq.
> Shindig's ProxyServlet should implement doPost
> ----------------------------------------------
>
> Key: SHINDIG-1555
> URL: https://issues.apache.org/jira/browse/SHINDIG-1555
> Project: Shindig
> Issue Type: Improvement
> Components: Java
> Affects Versions: 2.0.2
> Reporter: Stanton Sievers
> Labels: proxy
> Fix For: 3.0.0
>
> Original Estimate: 72h
> Remaining Estimate: 72h
>
> Shindig's ProxyServlet currently does not support HTTP POST. This
> improvement would be to augment the existing ProxyServlet to implement doPost
> as well as doGet.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira