utkarsharma2 commented on code in PR #39138:
URL: https://github.com/apache/airflow/pull/39138#discussion_r1593813122
##########
airflow/dag_processing/manager.py:
##########
@@ -728,6 +730,24 @@ def _add_callback_to_queue(self, request: CallbackRequest):
self._add_paths_to_queue([request.full_filepath], True)
Stats.incr("dag_processing.other_callback_count")
+ @provide_session
+ def _refresh_requested_filelocs(self, session=NEW_SESSION):
+ """Refresh filepaths from dag dir as requested by users via APIs."""
+ # Get values from DB table
+ requests = DagPriorityParsingRequests.get_requests()
+ for request in requests:
+ # Check if fileloc is in valid file paths. Parsing any
+ # filepaths can be a security issue.
Review Comment:
I think the code comment is sufficient here since we are taking measures to
prevent it and it should not affect user. Its added for developers.
--
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]
