You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2021/10/21 18:12:29 UTC

[GitHub] [superset] suddjian opened a new issue #17187: [SIP-75] Proposal for Embeddable Dashboards

suddjian opened a new issue #17187:
URL: https://github.com/apache/superset/issues/17187


   ## [SIP-75] Proposal for Embeddable Dashboards
   
   ### Motivation
   
   Embedding Superset dashboards into a third-party application is currently only possible if the dashboard is made public. We would like to add a feature to integrate dashboards into third-party applications ("Host Apps") with authentication handled by the third party.
   
   There should be no need to log in to an integrated dashboard if the user is already logged in to the Host App.
   
   Example use cases:
   - A storefront wants to offer engagement metrics to each of its vendors, integrated into the store management experience.
   - A bank wants to offer a financial wellness dashboard to account holders.
   
   Superset currently allows for a variety of valid strategies for granting API access, which can include various types of API Keys, depending on the deployment. Rather than force a specific API Key system into Superset, we’ve designed this spec to be agnostic to those details and depend on the existing security manager features as much as possible.
   
   ### Proposed Change
   
   1. Dashboard creator marks their dashboard as "embedded" in the dashboard UI. This design isn’t done yet, but it’ll likely be a modal or a new panel in the edit mode sidebar. Dashboard creator defines a list of allowed referrer domains.
   
   1. Introduce a new endpoint `POST /embedded-resource/`, which takes user information, returns a short-lived embed url including proof of being generated by Superset. This url will grant access to the resource.
   
   1. Example embed url: `https://example.org/superset/dashboard/123/embed?menu=0&title=0&rls=(team:a)&timestamp=1234567890&sig=blahblahblahblahblah`. It contains parameters controlling ui behavior, a timestamp so that the url will expire, and row-level-security parameters (could be encoded either as filter literals or as a RLS rule id). Lastly, Superset signs the url including the query and appends the result as the `sig` parameter.
   
   1. The Host App is responsible for loading the embed url in an iframe for the correct user and exposing the url to no one else.
   
   1. Browser loads the Superset url in the iframe. Superset backend checks the signature to ensure that it generated this url, that the url is not expired, and that the referrer is allowed.
   
   1. Superset issues a jwt to the embedded user for use as authentication in subsequent requests. This token includes user info, dashboard id, and rls info. This logically separates embedded users from regular users, and prevents their info from filling up tables in Superset.
   
   1. When a request is made using a jwt embedded auth token, security manager will use the information in the jwt to do its checks.
   
   ### New or Changed Public Interfaces
   
   `POST /embedded-resource/` endpoint
   
   ### New dependencies
   
   flask-jwt-extended for handling jwt embed tokens
   
   ### Migration Plan and Compatibility
   
   Will create new table `embedded_dashboards` containing a dashboard id reference, an allowlist of referrers, and 
   
   ### Rejected Alternatives
   
   OAuth login - requires users to sign in to the iframe, lower UX
   Generating a url or token in the Host App - better to keep generation on Superset side to avoid exposing implementation details, and so that we don’t have to define a specific API key spec
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] suddjian edited a comment on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
suddjian edited a comment on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-1082509877


   Heads up for anyone using this feature. Some time soon (after #19364 is merged) there will be an update pushing out a breaking change.
   
   Instead of using the *Dashboard* id, and any dashboard being embeddable via the sdk, dashboards will need to be configured to be embedded, and your app will need to use an *Embedded Dashboard* uuid. That uuid can be found through the dashboard's embed configuration screen. See that PR I mentioned for more info on that.
   
   I'll post here again when that update goes out. I don't anticipate further breaking changes after this update.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] idears-org commented on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
idears-org commented on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-1058971900


   @campino2k 
    I'm using helm to deploy on my cluster, and these setting are placed in my values.yaml
   ```
   # A dictionary of overrides to append at the end of superset_config.py - the name does not matter
   # WARNING: the order is not guaranteed
   configOverrides: 
     override:: |
       FEATURE_FLAGS = {
         "EMBEDDED_SUPERSET": True
       }
       ENABLE_PROXY_FIX = True
       PUBLIC_ROLE_LIKE = "Gamma"
       WTF_CSRF_ENABLED = False
   ```
   It will automatically set the `superset_config.py`
   There should be a way to set the same by using docker.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] gokhansari commented on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
gokhansari commented on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-1042907319


   Thank you @suddjian for the proposal and your commits. It'll be life saver.
   There is one thing that I would like to mention before you guys finalizing this feature. 
   
   [Custom Url Parameters](https://superset.apache.org/docs/installation/sql-templating/#available-macros) feature is one of the handy functionalities and It is not dispensable. When I check the master branch, for now It seems Embedded SDK only takes dashboardID parameter.
   To be able to use `{{ url_param('custom_variable') }}` and reference It in dataset SQL queries, I think there should be an other SDK parameter like `url_params: string` as we all used to use a lot of url parameters. 
   
   Example from doc:
   ```
   www.example.com/superset/sqllab?countrycode=ES&language=EN
   
   SELECT count(*)
   FROM ORDERS
   WHERE country_code = '{{ url_param('countrycode') }}' AND language_code = '{{ url_param('language') }}' 
   ```
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] suddjian commented on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
suddjian commented on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-1045047163


   Thanks for the comment, @gokhansari. Rather than using URL parameters, we can use the message channel shared by the SDK and Superset-within-iframe to accomplish this.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] campino2k commented on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
campino2k commented on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-1058988327


   Thanks. I inspected the config.py and found it also uses Environment Variables as overrides, so:
   
   ![grafik](https://user-images.githubusercontent.com/209926/156736195-fc1fcf46-cef5-4ffc-b88e-f5cc1d028a62.png)
   
   works like a charm.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] geido commented on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
geido commented on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-951315658


   It's definitely a +1. Great feature to have


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] suddjian commented on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
suddjian commented on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-953391794


   Thanks for the feedback @michael-s-molina!
   
   I've updated some of the implementation details to a flow that better supports expiration and revoking tokens - under this flow, refresh is handled by the frontend sdk and access can be revoked by not refreshing the token.
   
   Adding a wiki page is a great idea!
   
   Adding cross-window messaging is definitely possible, but out of scope for this SIP. It could certainly be added later and seems like a great idea to me!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] idears-org commented on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
idears-org commented on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-1046977422


   Hi @suddjian I'm trying to use the embedded SDK and integration the SDK into my project like this:
   ``` javascript
   embedDashboard({
         id: "3", // given by the Superset embedding UI
         supersetDomain: "http://localhost:62421/superset",
         mountPoint: document.getElementById("superset-container"), // any html element that can contain an iframe
         fetchGuestToken: () => {setTimeout(() => "a token from swagger ui", 10) },
         debug: true,
       });
   ```
   
   but the sdk inject an url http://localhost:62421/superset/dashboard/3/embedded in iframe which showing "page not found" am I doing wrong?
   
   <img width="582" alt="image" src="https://user-images.githubusercontent.com/13452500/154981894-6b8f0dd4-fd39-47ed-9495-c00bf8a02b56.png">
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] wemod123 commented on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
wemod123 commented on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-950128636


   I Had been looking for this feature for a while and almost do not have choice except grafana,
   We are embedding powerbi contents for statistical data analysis using powerbi embed API, and want to embed live data viz dashboard comes from direct query(powerbi direct query not powerful enough), if you are desing this kind of embed solution, powerbi embed API design may a good reference
   
   AND I am very looking forward this feature, Great Proposal
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] suddjian edited a comment on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
suddjian edited a comment on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-1082509877


   Heads up for anyone using this feature. Some time soon (after #19364 is merged) there will be an update pushing out a breaking change.
   
   Instead of using the *Dashboard* id, and any dashboard being embeddable via the sdk, dashboards will need to be configured to be embedded, and your app will need to use an *Embedded Dashboard* uuid. That uuid can be found through the dashboard options menu. See that PR I mentioned for more info on that.
   
   I'll post here again when that update goes out. I don't anticipate further breaking changes after this update.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] nytai edited a comment on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
nytai edited a comment on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-967489488


   This should be considered as part of this work 
   https://github.com/apache/superset/issues/17371


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] suddjian closed issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
suddjian closed issue #17187:
URL: https://github.com/apache/superset/issues/17187


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] suddjian commented on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
suddjian commented on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-949913071


   The usual authentication requirements for a Superset endpoint. If the caller is authenticated with the appropriate role, they are authorized. In Preset's case, a Preset API key would be used.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] nytai commented on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
nytai commented on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-967489488


   This should be considered as part of this work 
   https://github.com/apache/superset/issues/17187


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] nytai commented on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
nytai commented on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-949864339


   What are the security requirements of the `POST /embedded-resource/` endpoint? ie, how do we ensure that this call is coming from a trusted source? 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] suddjian edited a comment on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
suddjian edited a comment on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-1082509877


   Heads up for anyone using this feature. Some time soon (in a follow-up PR after #19364 is merged) there will be an update pushing out a breaking change.
   
   Instead of using the *Dashboard* id, and any dashboard being embeddable via the sdk, dashboards will need to be configured to be embedded, and your app will need to use an *Embedded Dashboard* uuid. That uuid can be found through the dashboard's embed configuration screen. See that PR I mentioned for more info on that.
   
   I'll post here again when that update goes out. I don't anticipate further breaking changes after this update.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] winterrobert commented on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
winterrobert commented on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-950214113


   This would be a great feature IMO.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] campino2k commented on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
campino2k commented on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-1058946684


   > Updates, I'm now able to use the embedded sdk by these settings: <img alt="image" width="468" src="https://user-images.githubusercontent.com/13452500/155241944-414889ef-54d3-40e4-898d-ee5b65674a19.png">
   
   Where did you put these values, @idears-org? I run superset from docker
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] suddjian commented on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
suddjian commented on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-1082509877


   Heads up for anyone using this feature. Some time soon (after #19364 is merged) there will be an update pushing out a breaking change.
   
   Instead of using the *Dashboard* id, and any dashboard being embeddable via the sdk, your app will need to use an *Embedded Dashboard* uuid that can be found through the dashboard options menu. See that PR I mentioned for more info on that.
   
   I'll post here again when that update goes out. I don't anticipate further breaking changes after this update.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] idears-org commented on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
idears-org commented on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-1047536295


   I'm using the kubernetes deployment, and already set the EMBEDDED_SUPERSET to True in pythonpath/superset_config.py by modify the configOverrides value in my values.yaml
   
   ![image](https://user-images.githubusercontent.com/13452500/155091360-f77f7065-ba58-4bac-a492-61d551dee97d.png)
   
   <img width="362" alt="image" src="https://user-images.githubusercontent.com/13452500/155091473-34ec709f-d267-45d3-802f-90728f0e4bad.png">
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] idears-org commented on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
idears-org commented on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-1048335746


   Updates, I'm now able to use the embedded sdk by these settings:
   <img width="468" alt="image" src="https://user-images.githubusercontent.com/13452500/155241944-414889ef-54d3-40e4-898d-ee5b65674a19.png">
   ![image](https://user-images.githubusercontent.com/13452500/155241996-2dd9b27a-7471-42d7-8cc5-23a2e25c4a8a.png)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] wemod123 edited a comment on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
wemod123 edited a comment on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-950128636


   I Had been looking for this feature for a while and almost do not have choice except grafana,
   We are embedding powerbi contents for statistical data analysis using powerbi embed API, and want to embed live data viz dashboard comes from direct query(powerbi direct query not powerful enough), if you are designing this kind of embed solution, powerbi embed API design may a good reference
   
   AND I am very looking forward this feature, Great Proposal
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] michael-s-molina commented on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
michael-s-molina commented on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-952239312


   @suddjian I have some broad questions to help with the SIP definition/scope:
   
   Do we have any plans to support cross-window communication? One common requirement is to allow interactions in the host app to impact the embedded dashboard or the other way around. Some examples:
   - The host app already has a filter bar and wishes to apply these filters to the dashboard
   - A user selects a slice of a pie chart and the host app shows a detailed report about the data
   
   Are we going to provide any sort of [client/toolkit](https://github.com/microsoft/PowerBI-JavaScript/) to our users to help with the embedding process?
   
   How are we going to handle URL and JWT expiration? Are we going to have specific endpoints for that?
   
   Should we also add a "How to Embed Superset" wiki page as one of the deliverables of this SIP? 
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] michael-s-molina edited a comment on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
michael-s-molina edited a comment on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-952239312


   @suddjian Thanks for writing this SIP. I have some broad questions to help with the SIP definition/scope:
   
   Do we have any plans to support cross-window communication? One common requirement is to allow interactions in the host app to impact the embedded dashboard or the other way around. Some examples:
   - The host app already has a filter bar and wishes to apply these filters to the dashboard
   - A user selects a slice of a pie chart and the host app shows a detailed report about the data
   
   Are we going to provide any sort of [client/toolkit](https://github.com/microsoft/PowerBI-JavaScript/) to our users to help with the embedding process?
   
   How are we going to handle URL and JWT expiration? Are we going to have specific endpoints for that?
   
   Should we also add a "How to Embed Superset" wiki page as one of the deliverables of this SIP? 
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] suddjian commented on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
suddjian commented on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-949935449


   An alternative to this is issuing a "signing key" to the dashboard creator when they mark their dashboard as embeddable. Then rather than making an API call to Superset to get the signed url, the Host App would craft the embed url and sign it, and Superset would verify the signature. The signing key acts similarly to an API key but maybe we don't need a separate SIP for that if it's only part of this feature.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] suddjian edited a comment on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
suddjian edited a comment on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-953391794


   Thanks for the feedback @michael-s-molina!
   
   I've updated some of the implementation details to a flow that better supports expiration and revoking tokens - under this flow, refresh is triggered by the frontend sdk and access can be revoked by not refreshing the token.
   
   Adding a wiki page is a great idea!
   
   Adding cross-window messaging is definitely possible, but out of scope for this SIP. It could certainly be added later and seems like a great idea to me!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] suddjian commented on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
suddjian commented on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-967489114


   This SIP is approved! Thanks for the discussion, all. Closing it now.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] suddjian edited a comment on issue #17187: [SIP-75] Proposal for Embeddable Dashboards

Posted by GitBox <gi...@apache.org>.
suddjian edited a comment on issue #17187:
URL: https://github.com/apache/superset/issues/17187#issuecomment-1082509877


   Heads up for anyone using this feature. Some time soon (after #19364 is merged) there will be an update pushing out a breaking change.
   
   Instead of using the *Dashboard* id, and any dashboard being embeddable via the sdk, dashboards will need to be configured to be embedded, and your app will need to use an *Embedded Dashboard* uuid that can be found through the dashboard options menu. See that PR I mentioned for more info on that.
   
   I'll post here again when that update goes out. I don't anticipate further breaking changes after this update.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org