Issue 49 in google-app-engine-oil: Session Cookie needs path attribute set.

[codesite...@google.com]

Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 49 by Martin.C.Simpson: Session Cookie needs path attribute set.
http://code.google.com/p/google-app-engine-oil/issues/detail?id=49

What steps will reproduce the problem?
1. Create a controller
2. Create a method on that controller that uses the self.session
3. Create a second controller
4. Try to see the cookie you set in the first controller, on the second
controller

What is the expected output? What do you see instead?
Expected output is that I see the same cookie for both controllers. I see
two cookies for each controller as the path is being set relative to the
controller. I.e one cookie for /controller1/ and one cookie for
/controller2/.

What version of the product are you using? On what operating system?
Mac OSX 10.6, Gaeo 0.3

Please provide any additional information below.
Fix:

In gaeo/session/memcache.py line 53:
cookie = '%s=%s' % (name, self._id)

should read:
cookie = '%s=%s; path=/' % (name, self._id)



--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

[codesite...@google.com]

Updates:
Status: Fixed

Comment #1 on issue 49 by ericsk: Session Cookie needs path attribute set.
http://code.google.com/p/google-app-engine-oil/issues/detail?id=49

Thanks for your fix :)