bingqin2 opened a new pull request, #72952:
URL: https://github.com/apache/airflow/pull/72952
`HttpToGCSOperator` documents `log_response` ("Log the response (default:
False)") and stores it, but `execute()` never reads it, so the response body is
never logged no matter what the user sets. This logs `response.text` after the
request when the flag is set, the same way `HttpOperator` does.
**Changes**
-
`providers/google/src/airflow/providers/google/cloud/transfers/http_to_gcs.py`:
log the response text when `log_response` is true
- `providers/google/tests/unit/google/cloud/transfers/test_http_to_gcs.py`:
`test_execute_logs_response_only_when_requested`, parametrized over both values
**Testing**
- `providers/google`: `tests/unit/google/cloud/transfers/test_http_to_gcs.py`
- mypy on the changed module, prek hooks on the changed files
---
##### Was generative AI tooling used to co-author this PR?
- [X] Yes (please specify the tool below)
Generated-by: Claude Code (Claude Fable 5.1) following [the
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions).
I reviewed and understand all changes; the tests were run locally as listed
above.
---
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
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]
