Reviewers: btlillie, Stanton, dev_shindig.apache.org,

Description:
For the shared token optin works better if the gadgets also share an
OAuth2Accessor.  Simple fix to rewrite the gadgetUri for accessors the
same way we do for tokens.

Please review this at http://codereview.appspot.com/6303083/

Affected files:
java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth2/BasicOAuth2Store.java


Index: java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth2/BasicOAuth2Store.java
===================================================================
--- java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth2/BasicOAuth2Store.java (revision 1350211) +++ java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth2/BasicOAuth2Store.java (working copy)
@@ -179,8 +179,10 @@
               gadgetUri, serviceName, user, scope });
     }

+    final String gadgetUri1 = this.getGadgetUri(gadgetUri, serviceName);
+
final OAuth2CallbackState state = new OAuth2CallbackState(this.stateCrypter);
-    state.setGadgetUri(gadgetUri);
+    state.setGadgetUri(gadgetUri1);
     state.setServiceName(serviceName);
     state.setUser(user);
     state.setScope(scope);
@@ -196,7 +198,7 @@
final OAuth2Token refreshToken = this.getToken(gadgetUri, serviceName, user, scope,
                 OAuth2Token.Type.REFRESH);

- final BasicOAuth2Accessor newAccessor = new BasicOAuth2Accessor(gadgetUri, serviceName, + final BasicOAuth2Accessor newAccessor = new BasicOAuth2Accessor(gadgetUri1, serviceName, user, scope, client.isAllowModuleOverride(), this, this.globalRedirectUri,
                 this.authority, this.contextRoot);
         newAccessor.setAccessToken(accessToken);


Reply via email to