self.length shield exception in httplib

[bsder]

BPO	1155638
Nosy	@birkenfeld, @birkenfeld

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = 'https://github.com/birkenfeld'
closed_at = <Date 2005-06-26.22:07:34.000>
created_at = <Date 2005-03-03.07:22:23.000>
labels = ['library']
title = 'self.length shield exception in httplib'
updated_at = <Date 2005-06-26.22:07:34.000>
user = 'https://bugs.python.org/bsder'

bugs.python.org fields:

activity = <Date 2005-06-26.22:07:34.000>
actor = 'georg.brandl'
assignee = 'georg.brandl'
closed = True
closed_date = None
closer = None
components = ['Library (Lib)']
creation = <Date 2005-03-03.07:22:23.000>
creator = 'bsder'
dependencies = []
files = []
hgrepos = []
issue_num = 1155638
keywords = []
message_count = 3.0
messages = ['24436', '24437', '24438']
nosy_count = 4.0
nosy_names = ['georg.brandl', 'georg.brandl', 'bsder', 'euske']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = None
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue1155638'
versions = ['Python 2.4']

[bsder]

Under certain conditions (I'm trying to open a
Shoutcast stream), I wind up with the following
exception from httplib:

 Traceback (most recent call last):
  File "/home/devel/lib/python2.4/threading.py", line
442, in __bootstrap
    self.run()
  File "avalanche.py", line 86, in run
    streamData = streamResponse.read(256)
  File "/home/devel/lib/python2.4/httplib.py", line
478, in read
    self.length -= len(s)
TypeError: unsupported operand type(s) for -=: 'str'
and 'int'

Normally, self.length has many shields of the form "if
self.length is None:"; however, self.length gets
initialize to _UNKNOWN which is the string "UNKNOWN"
rather than None. As such, all of the shields are useless.

Am I using a deprecated library or something? I'm
really surprised no one else has bumped into this.

[euske]

Logged In: YES
user_id=385990

I did bump into the same problem.
Apparently when I got HTTP/0.9 connection, self.length is
not initialized.
Inserting a line into l.362 at httplib.py (v2.4) seems to
solve this problem. I will also post a patch:

        if self.version == 9:
            self.chunked = 0
            self.will_close = 1
            self.msg = HTTPMessage(StringIO())
+            self.length = None
            return

[birkenfeld]

Logged In: YES
user_id=1188172

Thanks for the report! This is fixed as of Lib/httplib.py
r1.95, r1.94.2.1.