FIX Make sure that sessions only start when needed

tractorcow / silverstripe-dynamiccache

Simple on the fly caching of dynamic content for Silverstripe

39 stars 27 forks source link

FIX Make sure that sessions only start when needed #76

Closed dhensby closed 5 years ago

dhensby commented 6 years ago

Fixes #74

jonom commented 5 years ago

This seems to break the ability to login on a device where you don't already have an active session, because a PHPSESSID cookie isn't sent with the login form. After submitting the form I'm simply returned to the same form (unfilled). If I reverse this change I can log in fine. Is starting a session on every request a performance concern or what was the driver for this change?