Skip to content

Retry "Connection reset by peer"#4106

Merged
rouault merged 1 commit intoOSGeo:masterfrom
pjonsson:retry-reset-by-peer
Mar 27, 2024
Merged

Retry "Connection reset by peer"#4106
rouault merged 1 commit intoOSGeo:masterfrom
pjonsson:retry-reset-by-peer

Conversation

@pjonsson
Copy link
Contributor

Building the GDAL image can fail with:

Downloading https://cdn.proj.org/fr_ign_RAF18b.tif... (132 / 423)
Downloading https://cdn.proj.org/fr_ign_RAF20.tif... (133 / 423)
Downloading https://cdn.proj.org/fr_ign_RAGTBT2016.tif... (134 / 423)
Cannot open https://cdn.proj.org/fr_ign_RAGTBT2016.tif: OpenSSL SSL_read: Connection reset by peer, errno 104

so add the connection reset by peer
to the list of things that get a retry
time.

  • Tests added
  • Added clear title that can be used to generate release notes
  • Fully documented, including updating docs/source/*.rst for new API

Building the GDAL image can fail with:

Downloading https://cdn.proj.org/fr_ign_RAF18b.tif... (132 / 423)
Downloading https://cdn.proj.org/fr_ign_RAF20.tif... (133 / 423)
Downloading https://cdn.proj.org/fr_ign_RAGTBT2016.tif... (134 / 423)
Cannot open https://cdn.proj.org/fr_ign_RAGTBT2016.tif: OpenSSL SSL_read: Connection reset by peer, errno 104

so add the connection reset by peer
to the list of things that get a retry
time.
@pjonsson
Copy link
Contributor Author

I'm not sure if my patch fixes the problem described, or even how to test it, but the line seems to be in good company with the "Connection timed out" comparison.

@rouault
Copy link
Member

rouault commented Mar 27, 2024

Seems reasonable. As you also have interest in GDAL, would you mind issuing a similar patch to GDAL's CPLHTTPGetNewRetryDelay() method in port/cpl_http.cpp?

pjonsson added a commit to pjonsson/gdal that referenced this pull request Mar 27, 2024
This fixes the same issue as
OSGeo/PROJ#4106 fixes
in PROJ.
@rouault rouault merged commit 70d4d32 into OSGeo:master Mar 27, 2024
@pjonsson pjonsson deleted the retry-reset-by-peer branch March 27, 2024 12:08
@pjonsson pjonsson mentioned this pull request Mar 10, 2025
2 tasks
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