This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author demian.brecht
Recipients asvetlov, demian.brecht, docs@python, py.user, terry.reedy
Date 2013-02-28.07:24:47
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1362036287.73.0.0507515416224.issue16901@psf.upfronthosting.co.za>
In-reply-to
Content
@Terry: I don't think that the name change is unnecessary as the patch changes the semantics of the the LWP and Mozilla Cookie classes. In the patch, they no longer /are/ a subclass of a CookieJar, but they take a CookieJar object and implement the FileCookieProcessor interface in order to save/load/revert file-based cookies.

This solves the problem of an abstract base class (or at least, what was /intended/ to be an abstract base class prior to abcs: FileCookieJar) extending a concrete class, which doesn't make sense from an architectural standpoint. It also fixes the problem that people are encountering when attempting to instantiate a FileCookieJar object, it doesn't just patch something that's fundamentally broken. It could be my lack of experience with large scale OSS, but I'd prefer a /correct/ fix (especially to something that few actually care about and is seemingly not in heavy use) over a duct taped "just make it work" solution.

I absolutely agree, however, that this is a rather large change (as far as the cookiejar module goes anyway) and shouldn't be taken lightly. However, the cookiejar module /is/ in need of some love and I think that this takes a step to giving it that.

@py.user: My proposal was made to python-ideas. I just prefer gmane to Google Group for links.
History
Date User Action Args
2013-02-28 07:24:47demian.brechtsetrecipients: + demian.brecht, terry.reedy, asvetlov, docs@python, py.user
2013-02-28 07:24:47demian.brechtsetmessageid: <1362036287.73.0.0507515416224.issue16901@psf.upfronthosting.co.za>
2013-02-28 07:24:47demian.brechtlinkissue16901 messages
2013-02-28 07:24:47demian.brechtcreate