invalid range access during branch over http

Bug #62276 reported by Ollivier Robert
4
Affects Status Importance Assigned to Milestone
Bazaar
Fix Released
Low
Vincent Ladeuil

Bug Description

Running on OS X Tiger 10.4.7 with python 2.4.3 from DarwinPorts, cElementTree is installed.

567 [10:44] roberto@rron:Build/UP> time bzr branch http://bazaar-vcs.org/bzr/bzr.dev
bzr: ERROR: Invalid range access in http://bazaar-vcs.org/bzr/bzr.dev/.bzr/repository/knits/a7/test_reweave.py-20060302012326-a582932ad7879ccd.knit at 974.
bzr branch http://bazaar-vcs.org/bzr/bzr.dev 34.05s user 7.83s system 2% cpu 23:27.57 total

I have been getting this error since at least 0.9 a few months ago.

Related branches

Revision history for this message
Ollivier Robert (roberto-freebsd) wrote :
Revision history for this message
Ollivier Robert (roberto-freebsd) wrote :

Last 43 packets from the stream.

Revision history for this message
John A Meinel (jameinel) wrote :

Looking at the ethereal dump, I see that we make a request with

Range: bytes=0-853,974-1092

And the response seems to be multipart, and have:
Content-Type: multipart/byteranges; boundary="dFAeio52rA5c5-/w6C'c"

--dFAeio52rA5c5-/w6C'c

Content-Type: text/plain; charset=UTF-8

Content-Range: bytes 0-853/1093

Now, the actual content is gzipped binary data, so I don't know why it is return text/plain; charset=UTF-8, but the important bit is that there isn't a second chunk which contains:

--dFAeio52rA5c5-/w6C'c
Content-Type: text/plain; charset=UTF-8
Content-Range: bytes 974-1092/1093

I can confirm that bazaar-vcs.org is capable of returning the whole thing. So it seems like your proxy is restricting content to a single returned chunk, rather than returning the full request.

This seems to be a limitation of your proxy. Though it could be fixed in a similar way to bug 62029. Basically, we could detect that your http route is incapable of supporting a multi-range request, and change our requests to always be a single range, even though it is wasteful.

Changed in bzr:
importance: Untriaged → Low
status: Unconfirmed → Needs Info
Vincent Ladeuil (vila)
Changed in bzr:
assignee: nobody → v-ladeuil
status: Needs Info → In Progress
Revision history for this message
Vincent Ladeuil (vila) wrote :

A fix is available in the associated bzr.urllib.keepalive branch.

Revision history for this message
Vincent Ladeuil (vila) wrote :

New target 0.13

Changed in bzr:
status: In Progress → Fix Committed
Vincent Ladeuil (vila)
Changed in bzr:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.