Re: [PR] fix(security): Add input validation to cancel_query_id to prevent injection [superset]

["rusackas (via GitHub)" <gi...@apache.org>]

rusackas commented on PR #36722:
URL: https://github.com/apache/superset/pull/36722#issuecomment-4699030101

   Thanks @ColeMurray. Sorry your PR description seems to have been overwritten by the bot back when it was new. Anyway, there are a LOT of bot comments to take into consideration on this thread, but it looks like a solid defense-in-depth hardening with good test coverage across the engine specs. If you can resolve some of the issues the bots point out, if relevant, I think we can get this one to land soon. Sorry it didn't get more attention sooner.


-- 
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