8000 Patch NetHttpResponse to detect abrupt termination by wonderfly · Pull Request #279 · googleapis/google-http-java-client · GitHub
[go: up one dir, main page]

Skip to content
8000

Patch NetHttpResponse to detect abrupt termination #279

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.

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

Merged
merged 1 commit into from
Mar 17, 2015

Conversation

wonderfly
Copy link
Contributor

JDK's HttpUrlConnection implementation fails to notify clients when the
connection is lost when reading a response, e.g., the server gets restarted.
This changelist patches the NetHttpResponse to detect abrupt connection
termination when a fixed content length is available. In the case where
responses are chunk encoded, connection lost is well handled by JDK.

Also extend MockHttpUrlConnection for more testing functionalities.

  JDK's HttpUrlConnection implementation fails to notify clients when the
  connection is lost when reading a response, e.g., the server gets restarted.
  This changelist patches the NetHttpResponse to detect abrupt connection
  termination when a fixed content length is available. In the case where
  responses are chunk encoded, conne
A5F2
ction lost is well handled by JDK.

  Also extend MockHttpUrlConnection for more testing functionalities.
@wonderfly wonderfly assigned wonderfly and unassigned wonderfly Mar 13, 2015
wonderfly added a commit that referenced this pull request Mar 17, 2015
Patch NetHttpResponse to detect abrupt termination
@wonderfly wonderfly merged commit 94826ba into master Mar 17, 2015
@wonderfly wonderfly deleted the detectconnectionlost branch March 17, 2015 17:05
clundin25 pushed a commit to clundin25/google-http-java-client that referenced this pull request Aug 11, 2022
…s#279)

* Add Signer for impersonatied credentials

* add lint fixes

* back to the future

* Update oauth2_http/java/com/google/auth/oauth2/ImpersonatedCredentials.java

Co-Authored-By: Jeff Ching <chingor@google.com>
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.

1 participant
0