soooojinlee commented on issue #59310: URL: https://github.com/apache/airflow/issues/59310#issuecomment-3794459950
I treated this as a Jinja2 loader issue rather than a file‑open issue, so I added a zip‑aware loader in the task SDK. Regular paths keep `FileSystemLoader` behavior, and zip paths now use the same `split_template_path` validation to keep safety and behavior consistent. I also made sure searchpath order is preserved. I consolidated the tests into `task-sdk/tests/task_sdk/definitions/test_dag.py` to keep them in the SDK boundary. Coverage includes zipped DAG template loading, mixed searchpaths, ordering precedence, and zip path traversal blocking. Local tests passed. `open_maybe_zipped` lives in airflow‑core and is just a file‑open helper; this bug is about Jinja2’s template resolution path, so it wasn’t a good fit to reuse it. That’s why I fixed it in the loader instead. -- 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]
