[
https://issues.apache.org/jira/browse/SHINDIG-1660?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13149816#comment-13149816
]
[email protected] commented on SHINDIG-1660:
--------------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2824/
-----------------------------------------------------------
Review request for shindig.
Summary
-------
Update the ContainerConfig stack to allow for res://some-resource.txt and
file:///some-file.txt style values in container.js and automatically resolve
these references during initialization. This allows us to refactor
BlobCrypterSecurityTokenCodec to expect to be given the actual encryption key
rather than a key or resource pointer as it does currently.
This also opens the door for this functionality to be used for other
configuration which we'd like to load from external files (which should be
useful for third party implementers).
This addresses bug shindig-1660.
https://issues.apache.org/jira/browse/shindig-1660
Diffs
-----
http://svn.apache.org/repos/asf/shindig/trunk/config/container.js 1201745
http://svn.apache.org/repos/asf/shindig/trunk/java/common/src/main/java/org/apache/shindig/auth/BlobCrypterSecurityTokenCodec.java
1201745
http://svn.apache.org/repos/asf/shindig/trunk/java/common/src/main/java/org/apache/shindig/auth/DefaultSecurityTokenCodec.java
1201745
http://svn.apache.org/repos/asf/shindig/trunk/java/common/src/main/java/org/apache/shindig/common/util/ResourceLoader.java
1201745
http://svn.apache.org/repos/asf/shindig/trunk/java/common/src/main/java/org/apache/shindig/config/JsonContainerConfigLoader.java
1201745
http://svn.apache.org/repos/asf/shindig/trunk/java/common/src/test/java/org/apache/shindig/auth/BlobCrypterSecurityTokenCodecTest.java
1201745
http://svn.apache.org/repos/asf/shindig/trunk/java/common/src/test/java/org/apache/shindig/auth/DefaultSecurityTokenCodecTest.java
1201745
http://svn.apache.org/repos/asf/shindig/trunk/java/common/src/test/java/org/apache/shindig/config/JsonContainerConfigLoaderTest.java
1201745
http://svn.apache.org/repos/asf/shindig/trunk/java/common/src/test/resources/classpath-accessible-test-file.txt
PRE-CREATION
Diff: https://reviews.apache.org/r/2824/diff
Testing
-------
Updated/added tests. Ran build, verified unit tests and tested common
container.
Thanks,
Jesse
> Allow ContainerConfig stack to load property values from external resources
> and update BlobCrypterSecurityTokenCodec to use this new feature.
> ---------------------------------------------------------------------------------------------------------------------------------------------
>
> Key: SHINDIG-1660
> URL: https://issues.apache.org/jira/browse/SHINDIG-1660
> Project: Shindig
> Issue Type: Improvement
> Reporter: Jesse Ciancetta
>
> Update the ContainerConfig stack to allow for res://some-resource.txt and
> file:///some-file.txt style values in container.js and automatically resolve
> these references during initialization. This allows us to refactor
> BlobCrypterSecurityTokenCodec to expect to be given the actual encryption key
> rather than a key or resource pointer as it does currently.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
