Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#5952 closed defect (fixed)

Add retry after HTTP 500 error

Reported by: jpalmer Owned by: warmerdam
Priority: normal Milestone:
Component: default Version: unspecified
Severity: normal Keywords:
Cc:

Description

A follow-on to #5920/r28893. Fixes bug in retry logic loop not exiting if max re-entries is reached.

Attachments (1)

http_rentry.patch​ (1.3 KB ) - added by jpalmer 9 years ago.

Download all attachments as: .zip

Change History (3)

by jpalmer, 9 years ago

Attachment: http_rentry.patch​ added

comment:1 by Even Rouault, 9 years ago

Resolution: → fixed
Status: new → closed

I've fixed it differently with r29152 "Fix CPLHTTPFetch() retry logic (#5952)" since I think in your proposed patch, for example if you set GDAL_HTTP_MAX_RETRY to 1, no retry would be done despite the message and the sleep (said otherwise there would one GDAL_HTTP_MAX_RETRY - 1 retries)

comment:2 by jpalmer, 9 years ago

Ok thanks. Yip that's better logic.

Note: See TracTickets for help on using tickets.