Better session id needs to be rw fix #466

wants to merge 1 commit into

2 participants


Related to #460.

If the Session id attr is set, we should mark the Session as dirty.

@melo melo Mark Session as dirty if the id attr is modified
Signed-off-by: Pedro Melo <>
PerlDancer member

Closing this since it's a very old PR for something that isn't known to be an issue anymore.

@melo, if you disagree (and I trust you if you do), please reopen.

Thank you.

@xsawyerx xsawyerx closed this Aug 15, 2015

Hey, sorry, some time away from email… I don't even remember what triggered this on our code, and I don't remember seeing it with recent d2 releases, so I'm fine with this closed.


PerlDancer member

Thanks, @melo! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment