-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4135/
-----------------------------------------------------------

(Updated 2012-03-02 19:56:52.007316)


Review request for shindig.


Changes
-------

Updated diff per review from Stanton


Summary
-------

Simplify the SecurityToken.getActiveUrl implementation and definition.

If not set simply return null should be suffice.


This addresses bug SHINDIG-1719.
    https://issues.apache.org/jira/browse/SHINDIG-1719


Diffs (updated)
-----

  
trunk/java/common/src/test/java/org/apache/shindig/auth/BlobCrypterSecurityTokenTest.java
 1295758 
  trunk/java/common/src/main/java/org/apache/shindig/auth/SecurityToken.java 
1295758 
  
trunk/java/common/src/main/java/org/apache/shindig/auth/BasicSecurityToken.java 
1295758 
  
trunk/java/common/src/main/java/org/apache/shindig/auth/BlobCrypterSecurityToken.java
 1295758 

Diff: https://reviews.apache.org/r/4135/diff


Testing (updated)
-------

Updated unit test and run apps in common container


Thanks,

Henry

Reply via email to