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).

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.

-cbc


On Thu, Mar 22, 2018 at 5:05 PM, Levente Uzonyi <leves at caesar.elte.hu>
wrote:

> Hi Tobias,
>
> 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.
> WebClient still has no dependencies in the Trunk, so one should be able to
> safely unload it.
> But sure, we can just "nuke" HTTPSocket & co. in favor of WebClient.
>
> Levente
>
>
> On Thu, 22 Mar 2018, Tobias Pape wrote:
>
> Hi Levente
>>
>>
>> I really like the reuse-idea.
>> 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 :)
>>
>> Best regards
>>         -Tobias
>>
>>> On 22.03.2018, at 19:33, commits at source.squeak.org wrote:
>>>
>>> Levente Uzonyi uploaded a new version of Monticello to project The Trunk:
>>> http://source.squeak.org/trunk/Monticello-ul.679.mcz
>>>
>>> ==================== Summary ====================
>>>
>>> Name: Monticello-ul.679
>>> Author: ul
>>> Time: 22 March 2018, 7:32:51.372118 pm
>>> UUID: 7f5a3bf7-0cda-4200-b035-d9ce789f19d5
>>> Ancestors: Monticello-ul.678
>>>
>>> - enable #useSharedWebClientInstance by default when WebClient is present
>>>
>>> =============== Diff against Monticello-ul.678 ===============
>>>
>>> Item was changed:
>>>  ----- Method: MCHttpRepository class>>useSharedWebClientInstance (in
>>> category 'preferences') -----
>>>  useSharedWebClientInstance
>>>
>>>         <preference: 'Use shared WebClient instance'
>>>                 category: 'Monticello'
>>>                 description: 'When true, use a shared WebClient instance
>>> to speed up downloads from MCHttpRepositories. Requires WebClient to be
>>> present.'
>>>                 type: #Boolean>
>>> +       ^UseSharedWebClientInstance ifNil: [ Smalltalk hasClassNamed:
>>> #WebClient ]!
>>> -       ^UseSharedWebClientInstance ifNil: [ false ]!
>>>
>>>
>>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20180322/bad07fc0/attachment.html>