[
https://issues.apache.org/jira/browse/SHINDIG-1604?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13093096#comment-13093096
]
[email protected] commented on SHINDIG-1604:
--------------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1676/
-----------------------------------------------------------
Review request for shindig and li xu.
Summary
-------
Currently the code inject BasicAuthorityProvider to allow Shindig Java to be
deploy in non root environment.
However this is not necessary since the BasicAuthority is bind to Authority in
DefaultGuiceModule:
bind(Authority.class).toProvider(BasicAuthorityProvider.class);
we just need to inject Authority instance itself because we dont really need
multiple instances of Authority which is the main purpose of the Provider
injection.
Implementation of Authority such as BasicAuthority has to be immutable and
thread safe.
This addresses bug SHINDIG-1604.
https://issues.apache.org/jira/browse/SHINDIG-1604
Diffs
-----
trunk/java/common/src/main/java/org/apache/shindig/common/servlet/BasicAuthority.java
1162192
trunk/java/common/src/main/java/org/apache/shindig/common/servlet/BasicAuthorityProvider.java
1162192
trunk/java/common/src/test/java/org/apache/shindig/common/servlet/BasicAuthorityProviderTest.java
1162192
trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth/BasicOAuthStore.java
1162192
trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth/OAuthModule.java
1162192
trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/render/DefaultServiceFetcher.java
1162192
trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/DefaultConcatUriManager.java
1162192
trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/DefaultIframeUriManager.java
1162192
trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/DefaultJsUriManager.java
1162192
trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/DefaultProxyUriManager.java
1162192
trunk/java/social-api/src/main/java/org/apache/shindig/social/sample/spi/JsonDbOpensocialService.java
1162192
Diff: https://reviews.apache.org/r/1676/diff
Testing
-------
Updated the JUnit test case to reflect the change
Thanks,
Henry
> Inject Authority object instead of injecting the BasicAuthorityProvider itself
> ------------------------------------------------------------------------------
>
> Key: SHINDIG-1604
> URL: https://issues.apache.org/jira/browse/SHINDIG-1604
> Project: Shindig
> Issue Type: Improvement
> Components: Java
> Reporter: Henry Saputra
> Priority: Minor
> Attachments: SHINDIG-1604.patch
>
>
> Currently the code inject BasicAuthorityProvider to allow Shindig Java to be
> deploy in non root environment.
> However this is not necessary since the BasicAuthority is bind to Authority
> in DefaultGuiceModule:
> bind(Authority.class).toProvider(BasicAuthorityProvider.class);
> we just need to inject Authority instance itself because we dont really need
> multiple instances of Authority which is the main purpose of the Provider
> injection.
> Implementation of Authority such as BasicAuthority has to be immutable and
> thread safe.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
