[
https://issues.apache.org/jira/browse/SHINDIG-1651?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13137171#comment-13137171
]
[email protected] commented on SHINDIG-1651:
--------------------------------------------------------
bq. On 2011-10-27 00:08:02, Ryan Baxter wrote:
bq. > LGTM, I found the JIRA but can you attach the patch to it?
bq. > https://issues.apache.org/jira/browse/SHINDIG-1651
Committed revision 1189772.
- Ryan
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2572/#review2871
-----------------------------------------------------------
On 2011-10-27 12:06:49, Stanton Sievers wrote:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/2572/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2011-10-27 12:06:49)
bq.
bq.
bq. Review request for shindig and Brian Lillie.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. If a gadget uses OAuth but requires no other features that depend on the
"security-token" feature, a security token is not returned on the iframeurl
when it should be. This stems from the fact that ModulePrefs.getViewFeatures()
doesn't pick up the artificially added "security-token" feature that gets added
by the FeatureVisitor if the OAuthVisitors found oauth tags.
bq.
bq.
bq. This addresses bug SHINDIG-1651.
bq. https://issues.apache.org/jira/browse/SHINDIG-1651
bq.
bq.
bq. Diffs
bq. -----
bq.
bq.
http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/spec/ModulePrefs.java
1189151
bq.
http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/spec/ModulePrefsTest.java
1189151
bq.
bq. Diff: https://reviews.apache.org/r/2572/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq. Updated existing JUnits and added new JUnits.
bq.
bq.
bq. Thanks,
bq.
bq. Stanton
bq.
bq.
> Security tokens aren't returned on the iframeurl as part of a metadata
> response when the gadget uses OAuth
> ----------------------------------------------------------------------------------------------------------
>
> Key: SHINDIG-1651
> URL: https://issues.apache.org/jira/browse/SHINDIG-1651
> Project: Shindig
> Issue Type: Bug
> Components: Java
> Affects Versions: 3.0.0
> Reporter: Stanton Sievers
> Fix For: 3.0.0
>
> Attachments: ModulePrefsSecurityTokenPatch.txt
>
> Original Estimate: 1h
> Remaining Estimate: 1h
>
> If a gadget uses OAuth but requires no other features that depend on the
> "security-token" feature, a security token is not returned on the iframeurl
> when it should be. This stems from the fact that
> ModulePrefs.getViewFeatures() doesn't pick up the artificially added
> "security-token" feature that gets added by the FeatureVisitor if the
> OAuthVisitors found oauth tags.
--
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
