Skip to content

Honour the timeout while making the request#3849

Closed
urjitbhatia wants to merge 1 commit intogoogleapis:mainfrom
urjitbhatia:patch-1
Closed

Honour the timeout while making the request#3849
urjitbhatia wants to merge 1 commit intogoogleapis:mainfrom
urjitbhatia:patch-1

Conversation

@urjitbhatia
Copy link

The customized request method here isn't forwarding the timeout param to the base request object. This fix will enable proper timeout settings

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

The customized request method here isn't forwarding the timeout param to the base request object. This fix will enable proper timeout settings
@urjitbhatia urjitbhatia requested a review from a team as a code owner January 13, 2023 00:39
@sofisl
Copy link
Contributor

sofisl commented Jan 17, 2023

Hi @urjitbhatia, please open an issue here first, as this code is autogenerated from the library I linked.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants