Repository navigation
Same name cookies #51753
Description
| BPO | 7504 |
|---|---|
| Nosy | @terryjreedy, @amauryfa |
| Superseder | |
| Files |
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 = None
closed_at = <Date 2016-04-25.12:01:01.162>
created_at = <Date 2009-12-14.13:54:27.486>
labels = ['type-feature', 'library']
title = 'Same name cookies'
updated_at = <Date 2016-04-25.12:01:01.161>
user = 'https://bugs.python.org/chch'bugs.python.org fields:
activity = <Date 2016-04-25.12:01:01.161>
actor = 'berker.peksag'
assignee = 'none'
closed = True
closed_date = <Date 2016-04-25.12:01:01.162>
closer = 'berker.peksag'
components = ['Library (Lib)']
creation = <Date 2009-12-14.13:54:27.486>
creator = 'chch'
dependencies = []
files = ['15554', '15606']
hgrepos = []
issue_num = 7504
keywords = ['patch']
message_count = 8.0
messages = ['96375', '96595', '96596', '96597', '112727', '112753', '182764', '182817']
nosy_count = 4.0
nosy_names = ['terry.reedy', 'amaury.forgeotdarc', 'chch', 'mmelin']
pr_nums = []
priority = 'normal'
resolution = 'duplicate'
stage = 'resolved'
status = 'closed'
superseder = '1375011'
type = 'enhancement'
url = 'https://bugs.python.org/issue7504'
versions = ['Python 3.2']
Activity
chch commentedon Dec 14, 2009
After setting cookies with same name and different path only nearest to
root cookie is available.
Attached patch:
order.
chch commentedon Dec 19, 2009
Remake for patch due the requirements of
http://www.python.org/dev/patches/
amauryfa commentedon Dec 19, 2009
For this kind of report you should provide more information. Is this an
enhancement to the Cookie object? Why is it needed?
Do you really propose that users directly use the _multi attribute
directly (the leading _ suggest an internal attribute)?
Also, some documentation update is necessary (in Doc/library/cookie.rst)
chch commentedon Dec 19, 2009
This is fix for rare problem. If I set 2 cookies:
sid=pub.GHoBitAWLt, path="/"
sid=cab.S97jUfeihM, path="/cab"
All browsers in Cookie header send for any URL '/cab*':
sid=cab.S97jUfeihM; sid=pub.GHoBitAWLt
Current implementation always returns Cookie['sid']=pub.GHoBitAWLt - wrong!
My patch resolve this problem and Cookie['sid'] returns first (nearest/deepest in path) cookie value,
and also collects all same name cookies in dict Cookie._multi .
Name (_multi) may be wrong, and may be any other :o) .
Also I can update documentation, but my English is not so good ...
terryjreedy commentedon Aug 4, 2010
Unless you can quote something in the manual that says or implies that cookie.py should already have this feature, it looks like a feature request. Is it possible to add a new unittest? Prepare any new patches against py3k branch, now 'trunk'.
chch commentedon Aug 4, 2010
Cookie.py contain error!
mmelin commentedon Feb 23, 2013
FYI, this looks like the same issue as bpo-1375011
terryjreedy commentedon Feb 23, 2013
Which also has patches.