Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(252)

Issue 223090: Use ProxyUriManager in ProxyServlet (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
15 years, 10 months ago by johnfargo
Modified:
15 years, 10 months ago
Reviewers:
zhoresh, shindig.remailer
Base URL:
http://svn.apache.org/repos/asf/shindig/trunk/
Visibility:
Public.

Description

Swtiches ProxyServlet, by way of ProxyHandler (and ProxyBase, to a lesser extent) modifications, to use ProxyUriManager. The implementation follows the now-typical pattern of a UriManager's make/process methods.

Patch Set 1 #

Patch Set 2 : Adding missing PassthruManager class. #

Patch Set 3 : Sync'd and updated to latest. #

Total comments: 1

Patch Set 4 : Updated patch, removes helper class (already submitted). #

Messages

Total messages: 6
johnfargo
15 years, 10 months ago (2010-03-02 02:48:48 UTC) #1
johnfargo
Adding missing PassthruManager class.
15 years, 10 months ago (2010-03-02 02:55:06 UTC) #2
johnfargo
Sync'd and updated to latest.
15 years, 10 months ago (2010-03-03 19:45:19 UTC) #3
zhoresh
http://codereview.appspot.com/223090/diff/2001/3005 File java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/ProxyHandler.java (left): http://codereview.appspot.com/223090/diff/2001/3005#oldcode107 java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/ProxyHandler.java:107: if (request.getParameter(REFRESH_PARAM) != null) { Where is the handling ...
15 years, 10 months ago (2010-03-05 16:35:21 UTC) #4
johnfargo
Down in ProxyUriBase, with method translateRefreshStatus handling the logic involved in setting caching headers (taking ...
15 years, 10 months ago (2010-03-05 18:43:24 UTC) #5
johnfargo
15 years, 10 months ago (2010-03-09 01:57:14 UTC) #6
Updated patch, removes helper class (already submitted).
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b