Re: svn commit: r1495419 - in /subversion/trunk/subversion/libsvn_ra_serf: options.c ra_serf.h serf.c util.c

From: Philip Martin <philip.martin_at_wandisco.com>
Date: Fri, 28 Jun 2013 18:21:53 +0100

Philip Martin <philip.martin_at_wandisco.com> writes:

> Lieven Govaerts <svnlgo_at_mobsol.be> writes:
>
>>> It seems to be connected to mod_deflate. My server was loading
>>> mod_deflate (but not setting up any filters) and the import works with
>>> the client setting
>>>
>>> http-compression = no
>>>
>>> but fails with
>>>
>>> http-compression = yes
>>>
>>> Still investigating.
>>
>> Don't you need r1497551?
>
> I have that. If I configure nginx with "gzip = no" then lots of the
> regression tests now pass when run through nginx with my patch.

Some of the remaining failures are caused when
svn_ra_serf__get_inherited_props sends a REPORT request and receives:

  HTTP/1.1 411 Length Required\r
  Server: nginx/1.2.1\r
  Date: Fri, 28 Jun 2013 17:11:04 GMT\r
  Content-Type: text/html\r
  Content-Length: 180\r
  Connection: close\r
  \r
  <html>\r
  <head><title>411 Length Required</title></head>\r
  <body bgcolor="white">\r
  <center><h1>411 Length Required</h1></center>\r
  <hr><center>nginx/1.2.1</center>\r
  </body>\r
  </html>\r

I'd expect ra_serf to notice either the 411 or the Content-Type and
discard the body, but what happens is ra_serf attempts to parse the HTML
as XML.

-- 
Philip Martin | Subversion Committer
WANdisco | Non-Stop Data
www.wandisco.com

Received on 2013-06-28 19:22:33 CEST

Contemporary messages sorted: [ by date ] [ by thread ] [ by subject ] [ by author ] [ by messages with attachments ]