Add the Microsoft extension httponly to the
Cookie module.

The documentation change should say what the attribute does.  (It requests the the cookie be hidden from javascript, and available only to http requests.)

Sure, I have added some documentation to the patch.

File Added: python.diff

This is backwards-incompatible, no?  The behaviour of Morsel.set() changes (disallowing key="httponly") hence the behaviour of BaseCookie.__setitem__ changes.

Do you have a use case?

Anybody who sets a cookie with key="httponly" is likely in trouble.  I don't
know and can't check how the IE behaves in that case.  But disallowing this use
shouldn't hurt.

Use case: I would like to use the httponly attribute in Django.  I think it's
also useful for other web-frameworks.

I see.  That sounds reasonable, but I won't comment on whether it should be applied since this part of module Cookie didn't really make sense to me in the first place (I explain why in my comment of 2006-12-03 16:49 in http://python.org/sf/1372650).

Maybe you are right about the cookie module. I'm not so much into that.

But I just read that Firefox 3.0 Alpha 3 finally has support for the httponly attribute (see http://www.mozilla.org/projects/firefox/3.0a3/releasenotes/) so I think it's time that Python will also have the support.

Any word on this? I've tested the patch and it works. I'd like to use this in a Pylons application.

Any progress on this? This patch is extremely straightforward (only
three lines of code), and should not break existing code. 

The HttpOnly extension to cookies is now supported by IE, Firefox 3.0,
and Opera. 

This article explains why HttpOnly is a good way to make cross-site
scripting attacks significantly more difficult:

http://www.codinghorror.com/blog/archives/001167.htmllop

I'd really like to see this patch applied to Cookie.py.

Well, I'm sorry but this a feature request and must be delayed for
2.7/3.1, since 2.6/3.0 are now in the release candidate phase. :-(

(as for the patch, it would be nice if it added an unit test for the new
feature)

To be honest, I don't see any harm in adding this now, especially since
rc1 hasn't been released yet.

Well, if it's to be added then the patch should be updated to use reST.

I have updated the diff to use reST for the docs.  I removed the link to
MSDN from the reST docs because it is broken and I could not find the
article that it was intended to point to.  I also slightly re-worded the
paragraph describing httponly.  

I did not add any tests for the new feature as Antoine Pitrou requested,
because the test for Cookie only tests SimpleCookie. It does not test
expires, max-age, secure, or any of the other cookie attributes that
Cookie.py sets. Testing httponly (or any of the other cookie attributes)
would require rewriting most of the test.

The patch looks good to me and I will apply it soon if there are not
objections.

Rewriting of Cookie's tests should probably be another issue.

Ok. Applied in r66262.