rawwar commented on code in PR #51352:
URL: https://github.com/apache/airflow/pull/51352#discussion_r2123368898
##########
providers/http/src/airflow/providers/http/hooks/http.py:
##########
@@ -294,6 +312,24 @@ def run_and_check(
i.e. ``{'check_response': False}`` to avoid checking raising
exceptions on non 2XX
or 3XX status codes
"""
+ self.log.debug(
+ "Executing prepared request:\n"
+ "URL: %s\n"
+ "Method: %s\n"
+ "Headers: %s\n"
Review Comment:
I think, we should not print auth information here. Headers usually have
basic auth tokens as such. We need to filter them out or print partial values.
Along with that, can we just make a utility function that logs all this
information, given a request object? That's easier to reuse.
--
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: commits-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
