Review Board 1.7.22


Allow ContainerConfig stack to load property values from external resources and update BlobCrypterSecurityTokenCodec to use this new feature.

Review Request #2824 - Created Nov. 14, 2011 and submitted

Jesse Ciancetta
shindig-1660
Reviewers
shindig
shindig
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).
Updated/added tests.  Ran build, verified unit tests and tested common container.
Review request changed
Updated (Nov. 14, 2011, 8:46 p.m.)
Updated patch based on feedback.
Ship it!
Posted (Nov. 14, 2011, 9:24 p.m.)
Good stuff.  Thanks Jesse!
Ship it!
Posted (Nov. 15, 2011, 2:20 p.m.)
LGTM.  Jesse just following process here, could you attach the patch to the JIRA?  Then I will commit it.
  1. I'll give it a day or two to see if Paul might get a chance to add my Apache user id to the SVN commit access group.  If he does I'll go ahead and commit it myself otherwise I'll attach the patch to the JIRA ticket for you to commit.
  2. Sounds good to me
Ship it!
Posted (Nov. 15, 2011, 4:28 p.m.)
Committed r1202283.