Reviewers: johnfargo, zhoresh, shindig.remailer_gmail.com,
dev-remailer_shindig.apache.org, cool-shindig-committers_googlegroups.com,
Please review this at http://codereview.appspot.com/1822041/show
Affected files:
main/java/org/apache/shindig/gadgets/http/BasicHttpFetcher.java
main/java
adding tests
http://codereview.appspot.com/1822041/show
beautification
http://codereview.appspot.com/1822041/show
http://codereview.appspot.com/1822041/diff/9001/10004
File main/java/org/apache/shindig/gadgets/uri/UriUtils.java (right):
http://codereview.appspot.com/1822041/diff/9001/10004#newcode70
main/java/org/apache/shindig/gadgets/uri/UriUtils.java:70: // Headers
that the fetcher itself would like to f
http://codereview.appspot.com/1822041/diff/9001/10004
File main/java/org/apache/shindig/gadgets/uri/UriUtils.java (right):
http://codereview.appspot.com/1822041/diff/9001/10004#newcode70
main/java/org/apache/shindig/gadgets/uri/UriUtils.java:70: // Headers
that the fetcher itself would like to f
addressing comments
http://codereview.appspot.com/1822041/show
reverting basichttpfetcher
http://codereview.appspot.com/1822041/show
fixing comment
http://codereview.appspot.com/1822041/show
setting follow redirect to false for accel
http://codereview.appspot.com/1822041/show
Somehow the patch got out of sync w/ the current Shindig (chunk
mismatch) -- give it a quick update. Thx
http://codereview.appspot.com/1822041/diff/33001/34001
File main/java/org/apache/shindig/gadgets/http/HttpRequest.java (right):
http://codereview.appspot.com/1822041/diff/33001/34001#newcode
http://codereview.appspot.com/1822041/diff/33001/34001
File main/java/org/apache/shindig/gadgets/http/HttpRequest.java (right):
http://codereview.appspot.com/1822041/diff/33001/34001#newcode54
main/java/org/apache/shindig/gadgets/http/HttpRequest.java:54: // TODO:
Convert to Map> to allow multip
addressing comments
http://codereview.appspot.com/1822041/show
Hey Gagan:
Thanks for the responses. Brief comments inline, then continuing the review.
On Wed, Jul 14, 2010 at 11:20 PM, wrote:
>
> http://codereview.appspot.com/1822041/diff/33001/34001
> File main/java/org/apache/shindig/gadgets/http/HttpRequest.java (right):
>
> http://codereview.appspot.co
But there could be multiple header values for the same header, like:
Set-Cookie: NAME=value; domain=hello
Set-Cookie: NAME2=value2; domain=buffalo
Set header would just retain the last one (i hope thats why
HttpServletResponse has addHeader and setHeader functions)
Agreed w/ the rationale;
addressing comments
http://codereview.appspot.com/1822041/show
LGTM, looks great!
Patch applied, tested, and committed.
On Fri, Jul 16, 2010 at 12:57 PM, wrote:
> addressing comments
>
>
> http://codereview.appspot.com/1822041/show
>
16 matches
Mail list logo