You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@shindig.apache.org by zh...@gmail.com on 2010/04/09 22:08:57 UTC

Misc issues related to UriManagers (issue891047)

Reviewers: shindig.remailer_gmail.com, johnfargo,

Description:
Three minor changes attached:
- Add equal operator for ProxyUri so it can be easy to write tests
- Set schema null instead of empty string for IframeUriManager (for
empty string you get url '://')
- Change warning to info for Oath bad request

Please review this at http://codereview.appspot.com/891047/show

Affected files:
    
java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth/OAuthRequest.java
    
java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/ConcatUriManager.java
    
java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/DefaultIframeUriManager.java
    
java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/ProxyUriBase.java
    
java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/ProxyUriManager.java