Menu

#14 Failed fetches will never try again

0.9.7.0
closed
Network (1)
5
2014-08-05
2014-08-04
No

I recently noticed that when a fetch fails it seems to keep failing. I looked at the retry logic in imagemanager.cpp and I'm pretty sure that in the following code snippet you want less than rather than greater than.

imagemanager.cpp:109
else if ( failedFetches.contains(url) && failedFetches[url].secsTo(QDateTime::currentDateTime()) > FAILED_TIMEOUT_SECONDS )
{
//prevents spamming public servers when requests fail to return an image or server returns error code (busy/ivalid useragent etc)
qDebug() << "Ignored: " << url << " - last request failed less than 30 seconds ago";
}

Discussion

  • Brad Grimmett

    Brad Grimmett - 2014-08-05

    ouch. yep your right, should be less than.
    Fixing up now. Cheers

     
  • Brad Grimmett

    Brad Grimmett - 2014-08-05
    • labels: --> Network
    • status: unread --> open-accepted
    • assigned_to: Brad Grimmett
    • Group: 1.0.0.0 --> 0.9.7.0
     
  • Brad Grimmett

    Brad Grimmett - 2014-08-05

    Fixed in revision 54

     
  • Brad Grimmett

    Brad Grimmett - 2014-08-05
    • status: open-accepted --> closed
     

Log in to post a comment.

MongoDB Logo MongoDB