Graham Dumpleton
grahamd at dscpl.com.au
Mon Mar 28 23:23:59 EST 2005
Can you submit this at: http://issues.apache.org/jira/browse/MODPYTHON It will be lost in the mailing list archives otherwise. Thanks. Vivian De Smedt wrote .. > Dear mod_python developper, > > I just want notify you a small mistakes the Cookie module documentation. > The sample section use the get_cookie api instead of the get_cookies one. > In consequence the following section: > > from mod_python import apache, Cookie > > def handler(req): > > cookies = Cookie.get_cookie(req, Cookie.MarshalCookie, > secret='secret007') > if cookies.has_key('spam'): > spamcookie = cookies['spam'] > > req.write('Great, a spam cookie was found: %s\n' \ > % str(spamcookie)) > if type(spamcookie) is Cookie.MarshalCookie: > req.write('Here is what it looks like decoded: %s=%s\n' > % (spamcookie.name, spamcookie.value)) > else: > req.write('WARNING: The cookie found is not a \ > MarshalCookie, it may have been tapered with!') > > else: > > # MarshaCookie allows value to be any marshallable object > value = {'egg_count': 32, 'color': 'white'} > Cookie.add_cookie(req, Cookie.MarshalCookie('spam', value, > \ > 'secret007')) > req.write('Spam cookie not found, but we just set one!\n') > > return apache.OK > > should be rewritten: > > from mod_python import apache, Cookie > > def handler(req): > > cookies = Cookie.get_cookie*s*(req, Cookie.MarshalCookie, > secret='secret007') > if cookies.has_key('spam'): > spamcookie = cookies['spam'] > > req.write('Great, a spam cookie was found: %s\n' \ > % str(spamcookie)) > if type(spamcookie) is Cookie.MarshalCookie: > req.write('Here is what it looks like decoded: %s=%s\n' > % (spamcookie.name, spamcookie.value)) > else: > req.write('WARNING: The cookie found is not a \ > MarshalCookie, it may have been tapered with!') > > else: > > # MarshaCookie allows value to be any marshallable object > value = {'egg_count': 32, 'color': 'white'} > Cookie.add_cookie(req, Cookie.MarshalCookie('spam', value, > \ > 'secret007')) > req.write('Spam cookie not found, but we just set one!\n') > > return apache.OK > > Thanks for your nice module, > Vivian.
|