mirror/musikcube
1
0
Fork 0
mirror of https://github.com/clangen/musikcube.git synced 2024-12-28 15:16:43 +00:00
Code Issues Packages Projects Releases Wiki Activity

Tweaked retry count logic to be weighted based on failure type; if the

Browse Source 
server is overloaded let's allow for more retries with longer tries
between attempts.
This commit is contained in:
Casey Langen 2020-10-24 12:14:00 -07:00
parent a302837f1a
commit fbf8844c91
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/plugins/httpdatastream/HttpDataStream.cpp
View File

@ -396,8 +396,8 @@ void HttpDataStream::ResetFileHandles() {
void HttpDataStream::ThreadProc() {
if (this->curlEasy) {
static const int kMaxRetries = 3;
int retryCount = 0;
static const int kMaxRetries = 10;
int retryCount = 0; /* note: weighted based on failure type */
while (this->state != State::Downloaded && !this->interrupted) {
auto const curlCode = curl_easy_perform(this->curlEasy);
long httpStatusCode = 0;
@ -415,7 +415,7 @@ void HttpDataStream::ThreadProc() {
else {
if (httpStatusCode == 429) { /* too many requests */
this->state = State::Retrying;
++retryCount;
retryCount += 1;
sleepMs(5000);
}
else if ((httpStatusCode < 400 || httpStatusCode >= 500) && retryCount < kMaxRetries) {
@ -424,7 +424,7 @@ void HttpDataStream::ThreadProc() {
this->ResetFileHandles();
}
this->state = State::Retrying;
++retryCount;
retryCount += 2;
sleepMs(2000);
}
else {