feat: add X-Goog-Request-Reason header support#207
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces support for the requestReason property in BigQuery connections, enabling the X-Goog-Request-Reason header to be sent with HTTP requests. The changes include extracting the property in BQConnection, updating authorization methods in Oauth2Bigquery to propagate the reason and user agent, and modifying the HttpRequestTimeoutInitializer to set the headers. A new test case verifies the header's presence. Review feedback identifies a typo in a method name, suggests maintaining private visibility for the internal initializer class, recommends using a constant for the header name, and advises adding @Nullable annotations for clarity.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
siyumiG
left a comment
siyumiG
left a comment
There was a problem hiding this comment.
solve gemini-code-assit comment?
…c, use constant for header name, and add @nullable
3 participants
Internal branch for PR #205 to run CI with access to secrets.