rusackas commented on PR #43337: URL: https://github.com/apache/superset/pull/43337#issuecomment-5347961510
Good catch by codeant on the `escape_separator`/`unescape_separator` allowlist gap, that one's real. It's pre-existing on `master` though, not something you introduced, so opened #43345 to fix it separately rather than holding this up. Thanks for engaging with all the review feedback so thoroughly, this was a good one to work through! -- 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]
