164a7bc6-8686-4eed-9de5-3276f8858e6a
commented
5 years ago The current implementation of multipart/related in urllib triggers header defects even though the headers are valid:
[StartBoundaryNotFoundDefect(), MultipartInvariantViolationDefect()]
The example header is valid according to RFC 2387 (https://tools.ietf.org/html/rfc2387):
Content-Type: multipart/related; boundary="==="
Both defects are triggered by the fact that httplib only passes on headers to the underlying email parser, while the email parser assumes to receive a full message. The simple fix is to tell the underlying email parser that we are only passing the header: 0a89fc15c93c271eb08e46e2cda9a72adb97d633
The second issue is related, but independent: The underlying email parser checks if the parsed message is of type multipart by checking of the object "root" is of type list. As we only passed the header (and set headersonly=True), the check does makes no sense anymore at this point, creating a false positive: fdc7c47b77e330a36255fd00dc36accd72824e5b
vadmium
e888b79e-0010-4e57-af49-96f92d830757
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: ```python assignee = None closed_at = None created_at =
labels = ['3.7', 'type-bug', 'library']
title = 'multipart/related header causes false positive StartBoundaryNotFoundDefect and MultipartInvariantViolationDefect'
updated_at =
user = 'https://github.com/grische'
```
bugs.python.org fields:
```python
activity =
actor = 'ned.deily'
assignee = 'none'
closed = False
closed_date = None
closer = None
components = ['Library (Lib)']
creation =
creator = 'cschmidbauer'
dependencies = []
files = []
hgrepos = ['381']
issue_num = 36226
keywords = ['patch']
message_count = 5.0
messages = ['337395', '337401', '337444', '337661', '351114']
nosy_count = 6.0
nosy_names = ['barry', 'r.david.murray', 'martin.panter', 'maxking', 'tzickel', 'cschmidbauer']
pr_nums = ['12214']
priority = 'normal'
resolution = None
stage = 'patch review'
status = 'open'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue36226'
versions = ['Python 3.7']
```