[dev] [commits] Horde branch master updated. d2f9d03d6c60cd28c6f841cd251a8ac7415a6621

[Michael M Slusarz]

Quoting Michael Rubinsky <mrubinsk at horde.org>:

> Quoting Michael Rubinsky <mrubinsk at horde.org>:
>
>> Quoting Michael M Slusarz <slusarz at horde.org>:
>>
>>> The branch "master" has been updated.
>>> The following is a summary of the commits.
>>>
>>> from: 7869819cbfd94103184a404de1ba08b10b9d0b7c
>>>
>>> f285266 No longer used with the removal of framesets
>>> 01b53c3 Add binder for Horde_Http_Client
>>
>> If we are going to use the Injector for Http clients, should we use  
>> #createInstance() instead of #getInstance() since it's not out of  
>> the realm of possibilities that another bit of code has changed  
>> some default configuration? For example, the first code to request  
>> a http client might want to use POST instead of GET, but GET is the  
>> default behavior for Horde_Http_Request (and therefor  
>> Horde_Http_Client) objects.
>
> Eh, nevermind. I was misreading the code. Horde_Http_Client will set  
> those properties on each request. Sorry for the noise.

Seems as if you answered your own concerns... but the intent of the  
Http Client binder was simply to provide a copy of the client object  
that incorporates any global Horde configuration re: proxies.  No  
more, no less.

michael

-- 
___________________________________
Michael Slusarz [slusarz at horde.org]