http://codereview.appspot.com/1712043/diff/5001/6005
File
java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/AccelUriManager.java
(right):

http://codereview.appspot.com/1712043/diff/5001/6005#newcode65
java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/AccelUriManager.java:65:
String accelPath = config.getString(container, PROXY_PATH_PARAM);
Shouldn't it be accel_path?

http://codereview.appspot.com/1712043/diff/5001/6005#newcode67
java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/AccelUriManager.java:67:
return accelHost.equals(requestUri.getAuthority()) &&
I don't think we can promise that the authority will always match

http://codereview.appspot.com/1712043/show

Reply via email to