Re: [serf-dev] Re: ssl tunnel with basic authentication currently broken (was Re: Another crash in ra_serf in 1.8.0)

On Sat, Jun 22, 2013 at 3:26 PM, Lieven Govaerts <svnlgo_at_mobsol.be> wrote:
> On Sat, Jun 22, 2013 at 7:32 PM, Lieven Govaerts <svnlgo_at_mobsol.be> wrote:
>> Stefan,
>>
>> attached patch to serf 1.2.1 should solve this particular type of
>> crash you reported.
>>
>> The patch is made against a serf 1.2.x working copy as follows:
>> $ svn merge ^/trunk -c 1943,1944
>
> Unfortunately the attached patch was not entirely correct, even though
> for svn it seems to work ok, it breaks the new ssltunnel unit test.
>
> Attached an updated patch. I'll probably do some more testing this
> weekend, and commit any improvements to serf trunk.

So now: 1943, 1944, and 1946. I've reviewed the work and (by
inspection) it looks great.

And with no API changes :-D

Thanks!
-g
Received on 2013-06-23 03:20:53 CEST