Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

made underlying session.id accessible #17

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jvanasco
Copy link

server backed sessions have an .id property ( the value of the set-cookie's header )

for whatever reason(s), this was omitted from pyramid's ISession and consequently from pyramid_beaker, which makes identifying the session a pain ( one has to look at the cookies, and hope that the current session corresponds to the incoming cookie - which may or may not be the case if a cookie.id in request has been replaced with a new session.id in response )

this patch simply gives pyramid_beaker sessions an id property, which proxies a lookup to the id attribute of the underlying Beaker object.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant