[
https://issues.apache.org/jira/browse/SHINDIG-1681?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13183548#comment-13183548
]
[email protected] commented on SHINDIG-1681:
--------------------------------------------------------
bq. On 2012-01-10 18:48:26, Jesse Ciancetta wrote:
bq. >
http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container/container.js,
line 911
bq. > <https://reviews.apache.org/r/3180/diff/8/?file=67462#file67462line911>
bq. >
bq. > In the patch I was working on for moduleId's I took a different
approach here which I think you may be able to do as well. I think in my patch
-- any all the gadget tokens we had already fetched either by pre-load or
navigating a gadget (which would make them eligible for refresh) were cached in
the Service object -- so it seemed to me that knowing what tokens needed
refresh was a simple matter of getting the keys from the gadget token cache map
in the Service object.
bq. >
bq. > It looks like your keying the gadget token cache in the Service
object with the gadget spec URL + moduleId as well (or just a raw gadget spec
URL in the case of no moduleId) so it *seems* like that strategy might work for
you here too.
I agree this seems more straight forward, but I'm running out of time to work
on it. Do you think you can port that change from you patch on top of these
changes?
bq. On 2012-01-10 18:48:26, Jesse Ciancetta wrote:
bq. >
http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container.gadget/gadget_site.js,
line 202
bq. > <https://reviews.apache.org/r/3180/diff/8/?file=67458#file67458line202>
bq. >
bq. > This code (or something very similar) ends up repeated in a bunch of
different places -- it might be worth making a utility function somewhere which
takes the gadget URL and moduleId and returns the properly formatted string.
Thanks for pointing that out. I addressed all the related concerns with the
next patch I'm about to post.
- Dan
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3180/#review4298
-----------------------------------------------------------
On 2012-01-10 14:59:57, Dan Dumont wrote:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/3180/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2012-01-10 14:59:57)
bq.
bq.
bq. Review request for shindig, Henry Saputra, Ryan Baxter, li xu, Jesse
Ciancetta, and Stanton Sievers.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. Initial review of 1st change. Allowing common container to manage
container token refreshes. Also, refresh of gadget security tokens will now
wait for valid container security token before trying to refresh.
bq.
bq.
bq. This addresses bug SHINDIG-1681.
bq. https://issues.apache.org/jira/browse/SHINDIG-1681
bq.
bq.
bq. Diffs
bq. -----
bq.
bq.
http://svn.apache.org/repos/asf/shindig/trunk/content/samplecontainer/examples/commoncontainer/assembler.js
1222407
bq.
http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container.gadget/gadget_holder.js
1222407
bq.
http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container.gadget/gadget_site.js
1222407
bq.
http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container.util/constant.js
1222407
bq.
http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container.util/feature.xml
1222407
bq.
http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container.util/util.js
1222407
bq.
http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container/container.js
1222407
bq.
http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container/feature.xml
1222407
bq.
http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container/service.js
1222407
bq.
http://svn.apache.org/repos/asf/shindig/trunk/features/src/test/javascript/features/container/gadget_holder_test.js
1222407
bq.
http://svn.apache.org/repos/asf/shindig/trunk/features/src/test/javascript/features/container/service_test.js
1222407
bq.
http://svn.apache.org/repos/asf/shindig/trunk/java/common/src/main/java/org/apache/shindig/auth/AbstractSecurityToken.java
1222407
bq.
http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/GadgetsHandler.java
1222407
bq.
http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/GadgetsHandlerApi.java
1222407
bq.
http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/GadgetsHandlerService.java
1222407
bq.
http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/ModuleIdManager.java
PRE-CREATION
bq.
http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/ModuleIdManagerImpl.java
PRE-CREATION
bq.
http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/servlet/GadgetsHandlerServiceTest.java
1222407
bq.
http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/servlet/GadgetsHandlerTest.java
1222407
bq.
bq. Diff: https://reviews.apache.org/r/3180/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq. Tested code in a private container with some examples of setting no
refresh (ttl = 0) and setting an initial token (if it was written by jsp page
to avoid transaction) etc..
bq.
bq.
bq. Thanks,
bq.
bq. Dan
bq.
bq.
> Implement Container token Refresh and moduleId capability for gadget tokens.
> ----------------------------------------------------------------------------
>
> Key: SHINDIG-1681
> URL: https://issues.apache.org/jira/browse/SHINDIG-1681
> Project: Shindig
> Issue Type: New Feature
> Components: Java, Javascript
> Reporter: Dan Dumont
>
> Provides a refresh mechanism in the container for the container token and
> placeholders for implementing a gadget instance persistence mechanism that
> utilizes moduleIds in gadget security tokens.
--
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
