silverstripe / silverstripe-hybridsessions

Hybrid Cookie / DB Session store for Silverstripe

BSD 3-Clause "New" or "Revised" License

16 stars 21 forks source link

Backport #31 to SS3 #43

Closed brynwhyman closed 6 years ago

brynwhyman commented 6 years ago

31 (fixed in #32) was fixed for SS4 only.

Following an upgrade to PHP 7.1 on an SS3 site, users were unable to log in to the CMS. It could be related to this.

robbieaverill commented 6 years ago

I'll add it was reported that this only affects users with active DR enabled. It'd be good for someone to validate that the patch at #31 fixes the bug in an active DR environment before we backport it. Note that backporting the change won't be much work.

brynwhyman commented 6 years ago

TSP's PRE squad have confirmed that the issue described in #31 can be reproduced in SS 1.8.2

I've asked that they test the patch to ensure it fixes the issue. If it's not something they get to, we should still backport soon. Adding to backlog.

NightJar commented 6 years ago

Closed with #44