<body><div id="__MailbirdStyleContent" style="font-size: 10pt;font-family: Arial;color: #000000">
Hi Subbu!<div><br></div><div>+1</div><div><br></div><div>Those arbitrary number of retries did never work for me in Metacello either. There should some delay in between. The one you proposed here sounds good. And we have a virtually unlimited number of retries here. :-)</div><div><br></div><div>Best,</div><div>Marcel</div><div class="mb_sig"></div><blockquote class="history_container" type="cite" style="border-left-style:solid;border-width:1px; margin-top:20px; margin-left:0px;padding-left:10px;">
<p style="color: #AAAAAA; margin-top: 10px;">Am 02.04.2020 17:29:55 schrieb K K Subbu <kksubbu.ml@gmail.com>:</p><div style="font-family:Arial,Helvetica,sans-serif">Hi,<br><br>I have an alternate proposal to recover from 500 error on HTML help <br>topics. Instead of retrying 3 times [1], we just back off and retry only <br>when the document is needed next. If the server is busy then backing off <br>may be better than retries.<br><br>If a 'refresh' menu is available to force an update, then this changeset <br>is not needed.<br><br>[1] <br>http://forum.world.st/The-Inbox-HelpSystem-Core-ct-131-mcz-td5112713.html<br><br>Regards .. Subbu<br><br>On 02/04/20 3:15 PM, commits@source.squeak.org wrote:<br>> A new version of HelpSystem-Core was added to project The Inbox:<br>> http://source.squeak.org/inbox/HelpSystem-Core-kks.120.mcz<br>> <br>> ==================== Summary ====================<br>> <br>> Name: HelpSystem-Core-kks.120<br>> Author: kks<br>> Time: 2 April 2020, 8:45:13.607196 pm<br>> UUID: 33119c75-ba40-45d4-8ede-7086b7f4bc60<br>> Ancestors: HelpSystem-Core-mt.119<br>> <br>> Fetch a topic document afresh from the server if the previous fetch resulted in an 500 error.<br>> <br>> =============== Diff against HelpSystem-Core-mt.119 ===============<br>> <br>> Item was changed:<br>> ----- Method: HtmlHelpTopic>>document (in category 'accessing') -----<br>> document<br>> <br>> + (document isNil or: [document includesSubstring: '<title>500 Internal']) ifTrue: [<br>> + document := [<br>> + (HTTPSocket httpGet: self url accept: 'text/html') contents<br>> + ] on: Error do: [:err | err printString]].<br>> + ^document!<br>> - ^ document ifNil: [document :=<br>> - [<br>> - (HTTPSocket httpGet: self url accept: 'text/html') contents<br>> - ] on: Error do: [:err | err printString]]!<br>> <br>> <br><br><br></title></div></blockquote>
</div></body>