michael-s-molina commented on issue #27310: URL: https://github.com/apache/superset/issues/27310#issuecomment-1972032259
@sadpandajoe @dpgaspar https://github.com/apache/superset/pull/27117 removed the following deprecated endpoint: ``` @deprecated(new_target="/sqllab") def sqllab(self) -> FlaskResponse: ``` Which was registered in Superset's initialization: ``` appbuilder.add_view_no_menu(SqlLab) ``` `superset init` uses the registered views to dynamically generate the permissions, in that case, `can_sqllab` was generated from that entry and it does not exist anymore in a fresh installation of Superset. Searching for [can_sqllab](https://github.com/search?q=repo%3Aapache%2Fsuperset%20can_sqllab&type=code) we can still find many references to it in the code base, including the menu item that is missing. @dpgaspar I think the solution here would be to do a full revision of our permissions given all deprecated endpoints that were removed in 4.0 and update the code to only reference valid permissions. This was hard to spot before because the permissions are dynamically generated. -- 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]
