[jira] Commented: (SHINDIG-1460) Bug fix and improvement for common container: make gadgets can be rendered successfully with the new common container

["Christiaan Hees (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/SHINDIG-1460?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12927795#action_12927795 ] 

Christiaan Hees commented on SHINDIG-1460:
------------------------------------------

Can it also handle https? I see only a check for "http" in the patch for DefaultIframeUriManager.


> Bug fix and improvement for common container: make gadgets can be rendered successfully with the new common container
> ---------------------------------------------------------------------------------------------------------------------
>
>                 Key: SHINDIG-1460
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-1460
>             Project: Shindig
>          Issue Type: Improvement
>          Components: Javascript 
>    Affects Versions: 3.0.0
>            Reporter: Kai Feng Zhang
>             Fix For: 3.0.0
>
>         Attachments: common_container_1101.patch
>
>
> Working sample page on the new container, there are some issues we have to fix:
> 1. client side called "osapi.gadgets.metadata.get", but server side can't find that method, the key in the
> mapping of the rpc handler is "gadgets.metadata".
> 2. when creating the new metadatarequest in "util.js" in the new container, it used 'container':
> window.__CONTAINER, but window.__CONTAINER is undefined, we need provide one in the sample page or define it in the
> constants.js. Currently the patch provide one in the sample page.
> 3. after the above issues are  fixed, the returned iframeurl in metadata is something like this
> "//http://localhost:8080/gadgets/ifr?url=http%3A%2F%2Flocalhost%3A8080%2Fcontainer%2Fsample-pubsub-2-publisher.xml&
> amp;container=default&view=%25view%25&lang=%25lang%25&country=%25country%25&debug=%25debug%25&nocach
> e=%25nocache%25"
> The url began with two slashes. we have to remove them.
> One approach is changing it when generating the iframeurl on the server side, in the file of
> "DefaultIframeUriManager", another approach is changing it on the client side when setting it to the iframe
> src, in the file of "gadgets_holder.js".
> Currently the patch is based on the second approach.
> ----
> A new patch including following fixes and a sample page:
> 1. make gadgets.metadata.get can return the gadget's metadata
> 2. delete the slashes before the iframeurl
> 3. add pubsub-2 support in the common container
> 4. fix the security token issue if the gadget requires
> "security-token" feature.
> ----
> Latest patch:
> 1. change osapi.gadgets.metadata.get to osapi.gadgets.metadata
> in service.js
> 2. remove the leading "//" of the url to server side

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.