rusackas commented on issue #32961: URL: https://github.com/apache/superset/issues/32961#issuecomment-3643863114
This is a confirmed bug that bypasses the `is_managed_externally` protection. **Reproduced on:** 4.0.0, 4.0.2 **Security impact:** Dashboards marked as externally managed can still have charts added to them via the chart save flow, breaking the expected protection model. **Can anyone confirm if this persists on Superset 5.0+?** This would be a good issue for contribution - the fix would involve checking the target dashboard's `is_managed_externally` flag in the chart save logic before allowing addition. Adding `validation:validated` label since this has been confirmed by multiple users. -- 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]
