pierrejeambrun commented on issue #32360:
URL: https://github.com/apache/airflow/issues/32360#issuecomment-2410314978

   I just saw the Pull request associated to this and it makes me wonder. The 
actual `dag_id_pattern` is not used to do regexp matching, but db `ilike` 
macthing which does not leverage the standard regexp engine matcher but its own 
thing which is safer.
   
   For instance for postgres some informations are available here 
https://www.postgresql.org/docs/current/functions-matching.html#FUNCTIONS-LIKE, 
and as mentioned in the note just above seems safer from a security point of 
view. In opposition the paragraph right after highlights the `similar to 
regular expression` syntax, which is vulnerable.
   
   I am second guessing if we need to do something here.


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

Reply via email to