nytai commented on issue #14155: URL: https://github.com/apache/superset/issues/14155#issuecomment-820100578
I think the ideal fix here is to migrate this api to `/api/v1/. We already have an implementation for most major apis except for SQL Lab. Porting the sql_json endpoint over to a SQLRestApi implementation should be pretty straightforward since it's already a json api. Though, it could benefit from some refactor. @betodealmeida or @dpgaspar are either of you aware of any plans to add a sql endpoint to the api? @sewardgw disabling csrf protection on the endpoint _should_ be ok for now, as json apis are [not subject to csrf attacks](https://github.com/dxa4481/CORS) in most cases. -- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
