<div dir="ltr">I've finally started moving my use over to WebClient directly. I like the ability to tune the timeout (or turn it off) and the extended capabilities to modify response handling (some tricky rest clients didn't quite work out of the box).<div><br></div><div>I see no pressing reason not to move Trunk to using WebClient.  Maybe wait a release (or 3+, as seems to be common) to remove HTTPSocket, but move our usage away seems reasonable.</div><div><br></div><div>-cbc</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 22, 2018 at 5:05 PM, Levente Uzonyi <span dir="ltr"><<a href="mailto:leves@caesar.elte.hu" target="_blank">leves@caesar.elte.hu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Tobias,<br>
<br>
When WebClient was added, HTTPSocket was modified to use WebClient when it's present, but still work when it's not. I simply followed that path.<br>
WebClient still has no dependencies in the Trunk, so one should be able to<br>
safely unload it.<br>
But sure, we can just "nuke" HTTPSocket & co. in favor of WebClient.<span class="HOEnZb"><font color="#888888"><br>
<br>
Levente</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On Thu, 22 Mar 2018, Tobias Pape wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Levente<br>
<br>
<br>
I really like the reuse-idea.<br>
Also, WebClient is part of Squeak/Trunk since 4.5 I think. I think it is safe to assume that it's there. Let's rather completely deprecate HTTPSocket instead :)<br>
<br>
Best regards<br>
        -Tobias<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 22.03.2018, at 19:33, <a href="mailto:commits@source.squeak.org" target="_blank">commits@source.squeak.org</a> wrote:<br>
<br>
Levente Uzonyi uploaded a new version of Monticello to project The Trunk:<br>
<a href="http://source.squeak.org/trunk/Monticello-ul.679.mcz" rel="noreferrer" target="_blank">http://source.squeak.org/trunk<wbr>/Monticello-ul.679.mcz</a><br>
<br>
==================== Summary ====================<br>
<br>
Name: Monticello-ul.679<br>
Author: ul<br>
Time: 22 March 2018, 7:32:51.372118 pm<br>
UUID: 7f5a3bf7-0cda-4200-b035-d9ce78<wbr>9f19d5<br>
Ancestors: Monticello-ul.678<br>
<br>
- enable #useSharedWebClientInstance by default when WebClient is present<br>
<br>
=============== Diff against Monticello-ul.678 ===============<br>
<br>
Item was changed:<br>
 ----- Method: MCHttpRepository class>>useSharedWebClientInsta<wbr>nce (in category 'preferences') -----<br>
 useSharedWebClientInstance<br>
<br>
        <preference: 'Use shared WebClient instance'<br>
                category: 'Monticello'<br>
                description: 'When true, use a shared WebClient instance to speed up downloads from MCHttpRepositories. Requires WebClient to be present.'<br>
                type: #Boolean><br>
+       ^UseSharedWebClientInstance ifNil: [ Smalltalk hasClassNamed: #WebClient ]!<br>
-       ^UseSharedWebClientInstance ifNil: [ false ]!<br>
<br>
<br>
</blockquote></blockquote>
<br>
</div></div></blockquote></div><br></div>