Visitar URL original

Investigate connection reset in DNS helper · Issue #930 · googleapis/google-cloud-java · GitHub

Repository navigation

Investigate connection reset in DNS helper #930

Closed

Assignees

Labels

opened

on Apr 15, 2016

Without catching an IOException in the DNS helper handleBatch method, the testCreateChangeBatch causes connection reset.

mderka
added
api: dnsIssues related to the Cloud DNS API.
on Apr 15, 2016
mderka commented on Apr 15, 2016
mderka
on Apr 15, 2016
Author
Hint: The problem can be mitigated by simply appending \r\n to the response body. For example, change line 262 in LocalDnsHelper to:

Response response(String message) { try { return new Response(code, toJson(message) + "\r\n"); } catch (IOException e) { return Error.INTERNAL_ERROR.response("Error when generating JSON error response."); } }

Catching the exception is then not necessary. I did not make this change though. If this turns out to be needed, then the ResourceManagerHelper should be checked too as I think that DNS uses the same writer implementation. Perhaps these writers could be moved to core.
mziccard commented on Apr 18, 2016
mziccard
on Apr 18, 2016
Contributor
Fixed with #938
mziccard
closed this as completed on Apr 18, 2016
JustinBeckwith
assigned
mziccard
on Feb 1, 2021
github-actions
added a commit that references this issue on Jun 29, 2022
chore(main): release 2.1.5-SNAPSHOT (googleapis#930)
fb2b829
github-actions
added a commit that references this issue on Jul 14, 2022
build(deps): update dependency com.google.cloud:google-cloud-shared-c…
5e7a6db
github-actions
added 2 commits that reference this issue on Jul 27, 2022
deps: update dependency com.google.cloud:google-cloud-pubsub to v1.12…
413c06d
chore(main): release 3.6.1 (#933)
8586eb2
github-actions
added 2 commits that reference this issue on Aug 9, 2022
deps: update dependency com.google.cloud:google-cloud-storage to v2.1…
afc1b3f
chore(main): release 2.5.4 (#935)
dcf141b
github-actions
added 3 commits that reference this issue on Sep 15, 2022
chore(deps): update dependency com.google.cloud:libraries-bom to v26 (#…
5e1dec5
deps: update dependency io.grafeas:grafeas to v2.4.4 (#930)
905c9e4
chore(main): release 2.4.5 (#931)
69a8e7f
github-actions
added 2 commits that reference this issue on Oct 5, 2022
chore(main): release 2.11.0 (#922)
f41003a
chore(main): release 2.4.2-SNAPSHOT (#930)
37325cd
github-actions
added a commit that references this issue on Nov 16, 2022
build(deps): update dependency com.google.cloud:google-cloud-shared-c…
530ef82
suztomo
added 2 commits that reference this issue on Feb 1, 2023
chore(main): release 2.8.16 (#930)
5fe0d33
chore: upgrade native image downstream check to 22.3.0 (#930)
a6d95db
meltsufin
added 2 commits that reference this issue on Dec 22, 2025
deps: update dependency com.google.cloud:google-cloud-shared-dependen…
074c6c7
chore(main): release 3.7.5 (#932)
5767381
chingor13
added 2 commits that reference this issue on Jan 22, 2026
deps: update dependency org.graalvm.buildtools:junit-platform-native …
77285be
chore(main): release 2.13.0 (#921)
34868fe
suztomo
added 2 commits that reference this issue on Mar 11, 2026
deps: update dependency com.google.cloud:google-cloud-shared-dependen…
cd77490
chore(main): release 3.7.5 (googleapis#932)
55883b7
suztomo
added a commit that references this issue on Mar 23, 2026
chore(main): release 2.7.6 (googleapis#930)
0332de3
suztomo
added 2 commits that reference this issue on Mar 30, 2026
deps: update dependency com.google.cloud:google-cloud-storage to v2.8…
baa9df6
chore(main): release 0.124.5 (#929)
0a5adaa
lqiu96
added 2 commits that reference this issue on Apr 1, 2026
deps: update dependency com.google.cloud:google-cloud-storage to v2.8…
650472b
chore(main): release 0.124.5 (#929)
abb780a

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

Metadata

Assignees

mziccard

Labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests