davidnzhang commented on PR #68495: URL: https://github.com/apache/airflow/pull/68495#issuecomment-4785733209
@dabla Thanks for suggesting this - I just tested passing `response_type="bytes"` into the `run` method of the hook and can confirm that this works, as it completely bypasses the route where `DefaultResponseHandler` gets involved. That said, I'd argue the case for this change still stands, as the `DefaultResponseHandler` docstring already states it "returns JSON payload or content in bytes or response headers". Currently it fails to reach the bytes return path altogether for non-decodable binary responses, as `UnicodeDecodeError` is not caught by the suppress block. -- 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]
