[issue27466] [Copy from github user macartur] time2netscape missing comma

Changes by Senthil Kumaran : -- stage: patch review -> resolved ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue27466] [Copy from github user macartur] time2netscape missing comma

Senthil Kumaran added the comment: This is fixed in all versions of python. Thanks for the review. -- resolution: -> fixed status: open -> closed type: -> behavior ___ Python tracker _

[issue27466] [Copy from github user macartur] time2netscape missing comma

Roundup Robot added the comment: New changeset c613d8885054 by Senthil Kumaran in branch '2.7': [port to 2.7] - Issue #27466: Change time format returned by https://hg.python.org/cpython/rev/c613d8885054 -- ___ Python tracker

[issue27466] [Copy from github user macartur] time2netscape missing comma

Roundup Robot added the comment: New changeset 3356d7c57750 by Senthil Kumaran in branch '3.5': Issue #27466: Change time format returned by http.cookie.time2netscape, https://hg.python.org/cpython/rev/3356d7c57750 New changeset 324ade62c0f0 by Senthil Kumaran in branch 'default': [merge from 3.

[issue27466] [Copy from github user macartur] time2netscape missing comma

Senthil Kumaran added the comment: Fixed the redundant import as pointed by Evelyn on IRC. -- Added file: http://bugs.python.org/file43666/issue27466-v2.patch ___ Python tracker

[issue27466] [Copy from github user macartur] time2netscape missing comma

Robby Daigle added the comment: Thanks! -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.

[issue27466] [Copy from github user macartur] time2netscape missing comma

Senthil Kumaran added the comment: Here is the patch that fixes this issue. Patch includes unittests and NEWS entry. -- assignee: -> orsenthil keywords: +patch stage: -> patch review Added file: http://bugs.python.org/file43659/issue27466.patch ___

[issue27466] [Copy from github user macartur] time2netscape missing comma

Senthil Kumaran added the comment: Evelyn, Thanks for digging in. The code for time2netscape has been in the current form since it was originally introduced in 2004. If this is a bug, then it has been present since the introduction of the module. If anyone is using this module, they are prob

[issue27466] [Copy from github user macartur] time2netscape missing comma

Evelyn Mitchell added the comment: http://web.archive.org/web/19990128171928/http://www51.netscape.com/newsref/std/cookie_spec.html says: The date string is formatted as: Wdy, DD-Mon- HH:MM:SS GMT so the comma should be there. -- nosy: +efm ___

[issue27466] [Copy from github user macartur] time2netscape missing comma

New submission from Robby Daigle: For the time2netscape function. expected: Wed, DD-Mon- HH:MM:SS GMT got: Wed DD-Mon- HH:MM:SS GMT In Lib/http/cookiejar.py, line 116 suggests that the format should include a comma while line 123 does not include the comma in the formatted string. Wond