michael-s-molina commented on pull request #17337:
URL: https://github.com/apache/superset/pull/17337#issuecomment-964449300


   > Thanks for the response @michael-s-molina - I think as Superset moves into 
more organizations we should default to closed in all cases, so I would support 
the closed-sharing model. The open-sharing model is pretty dangerous - it 
becomes easy to create a link that's accessible to everyone in a company, which 
I believe wouldn't be a desired behavior by most larger organizations as a 
default behavior. In Superset, URL parameters, application state, and cache can 
contain highly sensitive information so I think we should shy away from the 
open-sharing model in all cases.
   
   @willbarrett These are good points. I agree that changing the default to the 
restricted model is more appropriate. I also think we should support the 
"Anyone with the key" model because we have some resources like public 
dashboards where we can benefit from it and we don't need the whole security 
configuration part from the user. I'll ping @dpgaspar to discuss this and 
increment the PR with these requirements.  Thank you so much again!


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to