bbovenzi closed pull request #37581: Fix rest API logging
URL: https://github.com/apache/airflow/pull/37581
--
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,
bbovenzi commented on PR #37581:
URL: https://github.com/apache/airflow/pull/37581#issuecomment-1965704065
Closing in favor of https://github.com/apache/airflow/pull/37734
I want to leave changes to variables+connections API and the `extra` field
as a separate effort
--
This is an
jscheffl commented on PR #37581:
URL: https://github.com/apache/airflow/pull/37581#issuecomment-1962562010
Looks good for me!
--
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
bbovenzi commented on PR #37581:
URL: https://github.com/apache/airflow/pull/37581#issuecomment-1958563244
Made `extra` a json object instead of a tuple. Added request body to
`extra`. And hiding sensitive fields for connections and variables:
vincbeck commented on PR #37581:
URL: https://github.com/apache/airflow/pull/37581#issuecomment-1956921252
LGTM
--
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
bbovenzi opened a new pull request, #37581:
URL: https://github.com/apache/airflow/pull/37581
Try to improve action logging in the REST API. Include
`api.{name_of_function}`. We should probably try to capture more information
for `extra` but audit log now reads like so:
