~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/transport/http/response.py

  • Committer: Vincent Ladeuil
  • Date: 2008-08-26 08:25:27 UTC
  • mto: (3668.1.1 trunk) (3703.1.1 trunk)
  • mto: This revision was merged to the branch mainline in revision 3669.
  • Revision ID: v.ladeuil+lp@free.fr-20080826082527-109yyxzc0u24oeel
Fix all calls to tempfile.mkdtemp to osutils.mkdtemp.

* bzrlib/transform.py:
(TransformPreview.__init__): Use osutils.mkdtemp instead of
tempfile.mkdtemp.

* bzrlib/tests/test_whitebox.py:
(MoreTests.test_relpath): Use osutils.mkdtemp instead of
tempfile.mkdtemp.

* bzrlib/tests/test_setup.py:
(TestSetup.test_build_and_install): Use osutils.mkdtemp instead of
tempfile.mkdtemp.

* bzrlib/tests/test_bundle.py:
(BundleTester.get_checkout): Use osutils.mkdtemp instead of
tempfile.mkdtemp.

* bzrlib/tests/blackbox/test_outside_wt.py:
(TestOutsideWT.test_cwd_log,
TestOutsideWT.test_diff_outside_tree): Use osutils.mkdtemp instead
of tempfile.mkdtemp.

* bzrlib/smart/repository.py:
(SmartServerRepositoryTarball._copy_to_tempdir): Use
osutils.mkdtemp instead of tempfile.mkdtemp.
(SmartServerRepositoryTarball._tarfile_response): Line too long.

* bzrlib/remote.py:
(RemoteRepository._copy_repository_tarball): Use osutils.mkdtemp
instead of tempfile.mkdtemp.

* bzrlib/osutils.py:
(_mac_mkdtemp): Add docstring.

* bzrlib/mail_client.py:
(ExternalMailClient.compose): Use osutils.mkdtemp instead of
tempfile.mkdtemp.

* bzrlib/diff.py:
(DiffFromTool.__init__): Use osutils.mkdtemp instead of
tempfile.mkdtemp.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2006 Michael Ellerman
 
1
# Copyright (C) 2006, 2007 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
22
22
"""
23
23
 
24
24
 
25
 
from bisect import bisect
 
25
import httplib
26
26
from cStringIO import StringIO
27
 
import re
28
 
 
29
 
from bzrlib import errors
30
 
from bzrlib.trace import mutter
31
 
 
32
 
 
33
 
class ResponseRange(object):
34
 
    """A range in a RangeFile-object."""
35
 
 
36
 
    __slots__ = ['_ent_start', '_ent_end', '_data_start']
37
 
 
38
 
    def __init__(self, ent_start, ent_end, data_start):
39
 
        self._ent_start = ent_start
40
 
        self._ent_end = ent_end
41
 
        self._data_start = data_start
42
 
 
43
 
    def __cmp__(self, other):
44
 
        """Compare this to other.
45
 
 
46
 
        We need this both for sorting, and so that we can
47
 
        bisect the list of ranges.
48
 
        """
49
 
        if isinstance(other, int):
50
 
            # Later on we bisect for a starting point
51
 
            # so we allow comparing against a single integer
52
 
            return cmp(self._ent_start, other)
53
 
        else:
54
 
            return cmp((self._ent_start, self._ent_end, self._data_start),
55
 
                       (other._ent_start, other._ent_end, other._data_start))
56
 
 
57
 
    def __str__(self):
58
 
        return "%s(%s-%s,%s)" % (self.__class__.__name__,
59
 
                                 self._ent_start, self._ent_end,
60
 
                                 self._data_start)
61
 
 
 
27
import rfc822
 
28
 
 
29
from bzrlib import (
 
30
    errors,
 
31
    trace,
 
32
    osutils,
 
33
    )
 
34
 
 
35
 
 
36
# A RangeFile expects the following grammar (simplified to outline the
 
37
# assumptions we rely upon).
 
38
 
 
39
# file: whole_file
 
40
#     | single_range
 
41
#     | multiple_range
 
42
 
 
43
# whole_file: [content_length_header] data
 
44
 
 
45
# single_range: content_range_header data
 
46
 
 
47
# multiple_range: boundary_header boundary (content_range_header data boundary)+
62
48
 
63
49
class RangeFile(object):
64
50
    """File-like object that allow access to partial available data.
65
51
 
66
 
    Specified by a set of ranges.
 
52
    All accesses should happen sequentially since the acquisition occurs during
 
53
    an http response reception (as sockets can't be seeked, we simulate the
 
54
    seek by just reading and discarding the data).
 
55
 
 
56
    The access pattern is defined by a set of ranges discovered as reading
 
57
    progress. Only one range is available at a given time, so all accesses
 
58
    should happen with monotonically increasing offsets.
67
59
    """
68
60
 
69
 
    def __init__(self, path, input_file):
 
61
    # in _checked_read() below, we may have to discard several MB in the worst
 
62
    # case. To avoid buffering that much, we read and discard by chunks
 
63
    # instead. The underlying file is either a socket or a StringIO, so reading
 
64
    # 8k chunks should be fine.
 
65
    _discarded_buf_size = 8192
 
66
 
 
67
    # maximum size of read requests -- used to avoid MemoryError issues in recv
 
68
    _max_read_size = 512 * 1024
 
69
 
 
70
    def __init__(self, path, infile):
 
71
        """Constructor.
 
72
 
 
73
        :param path: File url, for error reports.
 
74
        :param infile: File-like socket set at body start.
 
75
        """
70
76
        self._path = path
71
 
        self._pos = 0
72
 
        self._len = 0
73
 
        self._ranges = []
74
 
        self._data = input_file.read()
75
 
 
76
 
    def _add_range(self, ent_start, ent_end, data_start):
77
 
        """Add an entity range.
78
 
 
79
 
        :param ent_start: Start offset of entity
80
 
        :param ent_end: End offset of entity (inclusive)
81
 
        :param data_start: Start offset of data in data stream.
82
 
        """
83
 
        self._ranges.append(ResponseRange(ent_start, ent_end, data_start))
84
 
        self._len = max(self._len, ent_end)
85
 
 
86
 
    def _finish_ranges(self):
87
 
        self._ranges.sort()
88
 
 
89
 
    def read(self, size):
 
77
        self._file = infile
 
78
        self._boundary = None
 
79
        # When using multi parts response, this will be set with the headers
 
80
        # associated with the range currently read.
 
81
        self._headers = None
 
82
        # Default to the whole file of unspecified size
 
83
        self.set_range(0, -1)
 
84
 
 
85
    def set_range(self, start, size):
 
86
        """Change the range mapping"""
 
87
        self._start = start
 
88
        self._size = size
 
89
        # Set the new _pos since that's what we want to expose
 
90
        self._pos = self._start
 
91
 
 
92
    def set_boundary(self, boundary):
 
93
        """Define the boundary used in a multi parts message.
 
94
        
 
95
        The file should be at the beginning of the body, the first range
 
96
        definition is read and taken into account.
 
97
        """
 
98
        self._boundary = boundary
 
99
        # Decode the headers and setup the first range
 
100
        self.read_boundary()
 
101
        self.read_range_definition()
 
102
 
 
103
    def read_boundary(self):
 
104
        """Read the boundary headers defining a new range"""
 
105
        boundary_line = '\r\n'
 
106
        while boundary_line == '\r\n':
 
107
            # RFC2616 19.2 Additional CRLFs may precede the first boundary
 
108
            # string entity.
 
109
            # To be on the safe side we allow it before any boundary line
 
110
            boundary_line = self._file.readline()
 
111
 
 
112
        if boundary_line != '--' + self._boundary + '\r\n':
 
113
            # rfc822.unquote() incorrectly unquotes strings enclosed in <>
 
114
            # IIS 6 and 7 incorrectly wrap boundary strings in <>
 
115
            # together they make a beautiful bug, which we will be gracious
 
116
            # about here
 
117
            if (self._unquote_boundary(boundary_line) != 
 
118
                '--' + self._boundary + '\r\n'):
 
119
                raise errors.InvalidHttpResponse(
 
120
                    self._path,
 
121
                    "Expected a boundary (%s) line, got '%s'"
 
122
                    % (self._boundary, boundary_line))
 
123
 
 
124
    def _unquote_boundary(self, b):
 
125
        return b[:2] + rfc822.unquote(b[2:-2]) + b[-2:]
 
126
 
 
127
    def read_range_definition(self):
 
128
        """Read a new range definition in a multi parts message.
 
129
 
 
130
        Parse the headers including the empty line following them so that we
 
131
        are ready to read the data itself.
 
132
        """
 
133
        self._headers = httplib.HTTPMessage(self._file, seekable=0)
 
134
        # Extract the range definition
 
135
        content_range = self._headers.getheader('content-range', None)
 
136
        if content_range is None:
 
137
            raise errors.InvalidHttpResponse(
 
138
                self._path,
 
139
                'Content-Range header missing in a multi-part response')
 
140
        self.set_range_from_header(content_range)
 
141
 
 
142
    def set_range_from_header(self, content_range):
 
143
        """Helper to set the new range from its description in the headers"""
 
144
        try:
 
145
            rtype, values = content_range.split()
 
146
        except ValueError:
 
147
            raise errors.InvalidHttpRange(self._path, content_range,
 
148
                                          'Malformed header')
 
149
        if rtype != 'bytes':
 
150
            raise errors.InvalidHttpRange(self._path, content_range,
 
151
                                          "Unsupported range type '%s'" % rtype)
 
152
        try:
 
153
            # We don't need total, but note that it may be either the file size
 
154
            # or '*' if the server can't or doesn't want to return the file
 
155
            # size.
 
156
            start_end, total = values.split('/')
 
157
            start, end = start_end.split('-')
 
158
            start = int(start)
 
159
            end = int(end)
 
160
        except ValueError:
 
161
            raise errors.InvalidHttpRange(self._path, content_range,
 
162
                                          'Invalid range values')
 
163
        size = end - start + 1
 
164
        if size <= 0:
 
165
            raise errors.InvalidHttpRange(self._path, content_range,
 
166
                                          'Invalid range, size <= 0')
 
167
        self.set_range(start, size)
 
168
 
 
169
    def _checked_read(self, size):
 
170
        """Read the file checking for short reads.
 
171
 
 
172
        The data read is discarded along the way.
 
173
        """
 
174
        pos = self._pos
 
175
        remaining = size
 
176
        while remaining > 0:
 
177
            data = self._file.read(min(remaining, self._discarded_buf_size))
 
178
            remaining -= len(data)
 
179
            if not data:
 
180
                raise errors.ShortReadvError(self._path, pos, size,
 
181
                                             size - remaining)
 
182
        self._pos += size
 
183
 
 
184
    def _seek_to_next_range(self):
 
185
        # We will cross range boundaries
 
186
        if self._boundary is None:
 
187
            # If we don't have a boundary, we can't find another range
 
188
            raise errors.InvalidRange(self._path, self._pos,
 
189
                                      "Range (%s, %s) exhausted"
 
190
                                      % (self._start, self._size))
 
191
        self.read_boundary()
 
192
        self.read_range_definition()
 
193
 
 
194
    def read(self, size=-1):
90
195
        """Read size bytes from the current position in the file.
91
196
 
92
 
        Reading across ranges is not supported.
 
197
        Reading across ranges is not supported. We rely on the underlying http
 
198
        client to clean the socket if we leave bytes unread. This may occur for
 
199
        the final boundary line of a multipart response or for any range
 
200
        request not entirely consumed by the client (due to offset coalescing)
 
201
 
 
202
        :param size:  The number of bytes to read.  Leave unspecified or pass
 
203
            -1 to read to EOF.
93
204
        """
94
 
        # find the last range which has a start <= pos
95
 
        i = bisect(self._ranges, self._pos) - 1
96
 
 
97
 
        if i < 0 or self._pos > self._ranges[i]._ent_end:
98
 
            raise errors.InvalidRange(self._path, self._pos)
99
 
 
100
 
        r = self._ranges[i]
101
 
 
102
 
        # mutter('found range %s %s for pos %s', i, self._ranges[i], self._pos)
103
 
 
104
 
        if (self._pos + size - 1) > r._ent_end:
105
 
            raise errors.InvalidRange(self._path, self._pos)
106
 
 
107
 
        start = r._data_start + (self._pos - r._ent_start)
108
 
        end   = start + size
109
 
        # mutter("range read %d bytes at %d == %d-%d", size, self._pos,
110
 
        #         start, end)
111
 
        self._pos += (end-start)
112
 
        return self._data[start:end]
 
205
        if (self._size > 0
 
206
            and self._pos == self._start + self._size):
 
207
            if size == 0:
 
208
                return ''
 
209
            else:
 
210
                self._seek_to_next_range()
 
211
        elif self._pos < self._start:
 
212
            raise errors.InvalidRange(
 
213
                self._path, self._pos,
 
214
                "Can't read %s bytes before range (%s, %s)"
 
215
                % (size, self._start, self._size))
 
216
        if self._size > 0:
 
217
            if size > 0 and self._pos + size > self._start + self._size:
 
218
                raise errors.InvalidRange(
 
219
                    self._path, self._pos,
 
220
                    "Can't read %s bytes across range (%s, %s)"
 
221
                    % (size, self._start, self._size))
 
222
 
 
223
        # read data from file
 
224
        buffer = StringIO()
 
225
        limited = size
 
226
        if self._size > 0:
 
227
            # Don't read past the range definition
 
228
            limited = self._start + self._size - self._pos
 
229
            if size >= 0:
 
230
                limited = min(limited, size)
 
231
        osutils.pumpfile(self._file, buffer, limited, self._max_read_size)
 
232
        data = buffer.getvalue()
 
233
 
 
234
        # Update _pos respecting the data effectively read
 
235
        self._pos += len(data)
 
236
        return data
113
237
 
114
238
    def seek(self, offset, whence=0):
 
239
        start_pos = self._pos
115
240
        if whence == 0:
116
 
            self._pos = offset
 
241
            final_pos = offset
117
242
        elif whence == 1:
118
 
            self._pos += offset
 
243
            final_pos = start_pos + offset
119
244
        elif whence == 2:
120
 
            self._pos = self._len + offset
 
245
            if self._size > 0:
 
246
                final_pos = self._start + self._size + offset # offset < 0
 
247
            else:
 
248
                raise errors.InvalidRange(
 
249
                    self._path, self._pos,
 
250
                    "RangeFile: can't seek from end while size is unknown")
121
251
        else:
122
252
            raise ValueError("Invalid value %s for whence." % whence)
123
253
 
124
 
        if self._pos < 0:
125
 
            self._pos = 0
 
254
        if final_pos < self._pos:
 
255
            # Can't seek backwards
 
256
            raise errors.InvalidRange(
 
257
                self._path, self._pos,
 
258
                'RangeFile: trying to seek backwards to %s' % final_pos)
 
259
 
 
260
        if self._size > 0:
 
261
            cur_limit = self._start + self._size
 
262
            while final_pos > cur_limit:
 
263
                # We will cross range boundaries
 
264
                remain = cur_limit - self._pos
 
265
                if remain > 0:
 
266
                    # Finish reading the current range
 
267
                    self._checked_read(remain)
 
268
                self._seek_to_next_range()
 
269
                cur_limit = self._start + self._size
 
270
 
 
271
        size = final_pos - self._pos
 
272
        if size > 0: # size can be < 0 if we crossed a range boundary
 
273
            # We don't need the data, just read it and throw it away
 
274
            self._checked_read(size)
126
275
 
127
276
    def tell(self):
128
277
        return self._pos
129
278
 
130
279
 
131
 
class HttpRangeResponse(RangeFile):
132
 
    """A single-range HTTP response."""
133
 
 
134
 
    # TODO: jam 20060706 Consider compiling these regexes on demand
135
 
    _CONTENT_RANGE_RE = re.compile(
136
 
        '\s*([^\s]+)\s+([0-9]+)-([0-9]+)/([0-9]+)\s*$')
137
 
 
138
 
    def __init__(self, path, content_range, input_file):
139
 
        # mutter("parsing 206 non-multipart response for %s", path)
140
 
        RangeFile.__init__(self, path, input_file)
141
 
        start, end = self._parse_range(content_range, path)
142
 
        self._add_range(start, end, 0)
143
 
        self._finish_ranges()
144
 
 
145
 
    @staticmethod
146
 
    def _parse_range(range, path='<unknown>'):
147
 
        """Parse an http Content-range header and return start + end
148
 
 
149
 
        :param range: The value for Content-range
150
 
        :param path: Provide to give better error messages.
151
 
        :return: (start, end) A tuple of integers
152
 
        """
153
 
        match = HttpRangeResponse._CONTENT_RANGE_RE.match(range)
154
 
        if not match:
155
 
            raise errors.InvalidHttpRange(path, range,
156
 
                                          "Invalid Content-range")
157
 
 
158
 
        rtype, start, end, total = match.groups()
159
 
 
160
 
        if rtype != 'bytes':
161
 
            raise errors.InvalidHttpRange(path, range,
162
 
                    "Unsupported range type '%s'" % (rtype,))
163
 
 
164
 
        try:
165
 
            start = int(start)
166
 
            end = int(end)
167
 
        except ValueError, e:
168
 
            raise errors.InvalidHttpRange(path, range, str(e))
169
 
 
170
 
        return start, end
171
 
 
172
 
 
173
 
class HttpMultipartRangeResponse(RangeFile):
174
 
    """A multi-range HTTP response."""
175
 
    
176
 
    _CONTENT_TYPE_RE = re.compile(
177
 
        '^\s*multipart/byteranges\s*;\s*boundary\s*=\s*(.*?)\s*$')
178
 
    
179
 
    # Start with --<boundary>\r\n
180
 
    # and ignore all headers ending in \r\n
181
 
    # except for content-range:
182
 
    # and find the two trailing \r\n separators
183
 
    # indicating the start of the text
184
 
    # TODO: jam 20060706 This requires exact conformance
185
 
    #       to the spec, we probably could relax the requirement
186
 
    #       of \r\n, and use something more like (\r?\n)
187
 
    _BOUNDARY_PATT = (
188
 
        "^--%s(?:\r\n(?:(?:content-range:([^\r]+))|[^\r]+))+\r\n\r\n")
189
 
 
190
 
    def __init__(self, path, content_type, input_file):
191
 
        # mutter("parsing 206 multipart response for %s", path)
192
 
        # TODO: jam 20060706 Is it valid to initialize a
193
 
        #       grandparent without initializing parent?
194
 
        RangeFile.__init__(self, path, input_file)
195
 
 
196
 
        self.boundary_regex = self._parse_boundary(content_type, path)
197
 
 
198
 
        for match in self.boundary_regex.finditer(self._data):
199
 
            ent_start, ent_end = HttpRangeResponse._parse_range(match.group(1), path)
200
 
            self._add_range(ent_start, ent_end, match.end())
201
 
 
202
 
        self._finish_ranges()
203
 
 
204
 
    @staticmethod
205
 
    def _parse_boundary(ctype, path='<unknown>'):
206
 
        """Parse the Content-type field.
207
 
        
208
 
        This expects a multipart Content-type, and returns a
209
 
        regex which is capable of finding the boundaries
210
 
        in the multipart data.
211
 
        """
212
 
        match = HttpMultipartRangeResponse._CONTENT_TYPE_RE.match(ctype)
213
 
        if not match:
214
 
            raise errors.InvalidHttpContentType(path, ctype,
215
 
                    "Expected multipart/byteranges with boundary")
216
 
 
217
 
        boundary = match.group(1)
218
 
        # mutter('multipart boundary is %s', boundary)
219
 
        pattern = HttpMultipartRangeResponse._BOUNDARY_PATT
220
 
        return re.compile(pattern % re.escape(boundary),
221
 
                          re.IGNORECASE | re.MULTILINE)
222
 
 
223
 
 
224
 
def _is_multipart(content_type):
225
 
    return content_type.startswith('multipart/byteranges;')
226
 
 
227
 
 
228
 
def handle_response(url, code, headers, data):
229
 
    """Interpret the code & headers and return a HTTP response.
230
 
 
231
 
    This is a factory method which returns an appropriate HTTP response
232
 
    based on the code & headers it's given.
 
280
def handle_response(url, code, msg, data):
 
281
    """Interpret the code & headers and wrap the provided data in a RangeFile.
 
282
 
 
283
    This is a factory method which returns an appropriate RangeFile based on
 
284
    the code & headers it's given.
233
285
 
234
286
    :param url: The url being processed. Mostly for error reporting
235
287
    :param code: The integer HTTP response code
236
 
    :param headers: A dict-like object that contains the HTTP response headers
 
288
    :param msg: An HTTPMessage containing the headers for the response
237
289
    :param data: A file-like object that can be read() to get the
238
290
                 requested data
239
291
    :return: A file-like object that can seek()+read() the 
240
292
             ranges indicated by the headers.
241
293
    """
242
 
 
243
 
    if code == 206:
244
 
        try:
245
 
            content_type = headers['Content-Type']
246
 
        except KeyError:
247
 
            raise errors.InvalidHttpContentType(url, '',
248
 
                msg = 'Missing Content-Type')
249
 
 
250
 
        if _is_multipart(content_type):
 
294
    rfile = RangeFile(url, data)
 
295
    if code == 200:
 
296
        # A whole file
 
297
        size = msg.getheader('content-length', None)
 
298
        if size is None:
 
299
            size = -1
 
300
        else:
 
301
            size = int(size)
 
302
        rfile.set_range(0, size)
 
303
    elif code == 206:
 
304
        content_type = msg.getheader('content-type', None)
 
305
        if content_type is None:
 
306
            # When there is no content-type header we treat the response as
 
307
            # being of type 'application/octet-stream' as per RFC2616 section
 
308
            # 7.2.1.
 
309
            # Therefore it is obviously not multipart
 
310
            content_type = 'application/octet-stream'
 
311
            is_multipart = False
 
312
        else:
 
313
            is_multipart = (msg.getmaintype() == 'multipart'
 
314
                            and msg.getsubtype() == 'byteranges')
 
315
 
 
316
        if is_multipart:
251
317
            # Full fledged multipart response
252
 
            return HttpMultipartRangeResponse(url, content_type, data)
 
318
            rfile.set_boundary(msg.getparam('boundary'))
253
319
        else:
254
320
            # A response to a range request, but not multipart
255
 
            try:
256
 
                content_range = headers['Content-Range']
257
 
            except KeyError:
 
321
            content_range = msg.getheader('content-range', None)
 
322
            if content_range is None:
258
323
                raise errors.InvalidHttpResponse(url,
259
324
                    'Missing the Content-Range header in a 206 range response')
260
 
            return HttpRangeResponse(url, content_range, data)
261
 
    elif code == 200:
262
 
        # A regular non-range response, unfortunately the result from
263
 
        # urllib doesn't support seek, so we wrap it in a StringIO
264
 
        tell = getattr(data, 'tell', None)
265
 
        if tell is None:
266
 
            return StringIO(data.read())
267
 
        return data
268
 
    elif code == 404:
269
 
        raise errors.NoSuchFile(url)
 
325
            rfile.set_range_from_header(content_range)
 
326
    else:
 
327
        raise errors.InvalidHttpResponse(url,
 
328
                                         'Unknown response code %s' % code)
270
329
 
271
 
    raise errors.InvalidHttpResponse(url, "Unknown response code %s" % (code,))
 
330
    return rfile
272
331