Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

POST/PUT data can hang indefinitely and ignore timeouts #484

Closed
chingor13 opened this issue Sep 27, 2018 · 0 comments
Closed

POST/PUT data can hang indefinitely and ignore timeouts #484

chingor13 opened this issue Sep 27, 2018 · 0 comments
Assignees
@chingor13
Labels
priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.

Comments

@chingor13
Copy link
Collaborator

It's possible that the local system's output stream blocks when writing data. We should add the ability to set a timeout on the socket write portion. This portion of the request ignores the connect and read timeout settings.

Environment details

  • OS: Linux
  • Java version: 7, 8
  • google-http-java-client version: 1.23.0

Steps to reproduce

Example test: https://github.com/chingor13/http-write-timeout

Any additional information below

A few customers are seeing this issue.
@chingor13 chingor13 added type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. labels Sep 27, 2018
@chingor13 chingor13 self-assigned this Sep 27, 2018
@chingor13 chingor13 mentioned this issue Sep 27, 2018
Merged
2 tasks
@JustinBeckwith JustinBeckwith added 🚨 This issue needs some love. and removed 🚨 This issue needs some love. labels Oct 2, 2018
@JustinBeckwith JustinBeckwith added 🚨 This issue needs some love. and removed 🚨 This issue needs some love. labels Oct 12, 2018
@JustinBeckwith JustinBeckwith added 🚨 This issue needs some love. and removed 🚨 This issue needs some love. labels Oct 22, 2018
@JustinBeckwith JustinBeckwith added 🚨 This issue needs some love. and removed 🚨 This issue needs some love. labels Nov 1, 2018
@yoshi-automation yoshi-automation removed the 🚨 This issue needs some love. label Apr 6, 2020
yoshi-automation added a commit that referenced this issue Apr 14, 2020
@yoshi-automation
build(java): set GOOGLE_CLOUD_PROJECT env for samples/integration tes…
4bc56ae 
…ts (#484)

* build(java): set GOOGLE_CLOUD_PROJECT env variable for samples/integration tests

* ci: use java-docs-samples-testing for sample tests

Source-Author: Jeff Ching <[email protected]>
Source-Date: Mon Apr 13 16:24:21 2020 -0700
Source-Repo: googleapis/synthtool
Source-Sha: 52638600f387deb98efb5f9c85fec39e82aa9052
Source-Link: googleapis/synthtool@5263860
yoshi-automation added a commit that referenced this issue Apr 22, 2020
@yoshi-automation
build(java): set GOOGLE_CLOUD_PROJECT env for samples/integration tes…
ecfb9ae 
…ts (#484)

* build(java): set GOOGLE_CLOUD_PROJECT env variable for samples/integration tests

* ci: use java-docs-samples-testing for sample tests

Source-Author: Jeff Ching <[email protected]>
Source-Date: Mon Apr 13 16:24:21 2020 -0700
Source-Repo: googleapis/synthtool
Source-Sha: 52638600f387deb98efb5f9c85fec39e82aa9052
Source-Link: googleapis/synthtool@5263860
yoshi-automation added a commit that referenced this issue Apr 25, 2020
@yoshi-automation
build(java): set GOOGLE_CLOUD_PROJECT env for samples/integration tes…
ece4f10 
…ts (#484)

* build(java): set GOOGLE_CLOUD_PROJECT env variable for samples/integration tests

* ci: use java-docs-samples-testing for sample tests

Source-Author: Jeff Ching <[email protected]>
Source-Date: Mon Apr 13 16:24:21 2020 -0700
Source-Repo: googleapis/synthtool
Source-Sha: 52638600f387deb98efb5f9c85fec39e82aa9052
Source-Link: googleapis/synthtool@5263860
chingor13 pushed a commit that referenced this issue Apr 27, 2020
@yoshi-automation
chore: update common templates (#1035)
1624d55 
* changes without context

        autosynth cannot find the source of changes triggered by earlier changes in this
        repository, or by version upgrades to tools such as linters.

* ci: add dependency list completeness check (#490)

Source-Author: Stephanie Wang <[email protected]>
Source-Date: Mon Apr 13 18:30:27 2020 -0400
Source-Repo: googleapis/synthtool
Source-Sha: 3df869dd6eb546ef13beeb7a9efa6ee0226afafd
Source-Link: googleapis/synthtool@3df869d

* build(java): set GOOGLE_CLOUD_PROJECT env for samples/integration tests (#484)

* build(java): set GOOGLE_CLOUD_PROJECT env variable for samples/integration tests

* ci: use java-docs-samples-testing for sample tests

Source-Author: Jeff Ching <[email protected]>
Source-Date: Mon Apr 13 16:24:21 2020 -0700
Source-Repo: googleapis/synthtool
Source-Sha: 52638600f387deb98efb5f9c85fec39e82aa9052
Source-Link: googleapis/synthtool@5263860
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chingor13 chingor13

Labels
priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@chingor13 @JustinBeckwith @yoshi-automation