cnzz/memcached-session-manager

By default, perform session backup asynchronously (change default for sessionBackupAsync from false to true)

Closed this issue · 1 comments

Right now, session backup is done synchronously by default. This default 
should be changed to async.

Background:
The synchronous backup was made the default as I thought, that one might 
notice backup failures, relocate the session and send a cookie to the 
client. However, in most cases, the response is already comitted when the 
backup is done, so the session should _not_ get relocated, as the cookie 
would not reach the client. Instead, failures during backup should be 
accepted and regarded as _really_ unprobable. They can be regarded 
unprobable, because at the start of a request it's checked if session 
relocation is required or not. Only in this case, a session is relocated, 
because in this case we _definitely_ can send a cookie to the browser.

Original issue reported on code.google.com by martin.grotzke on 23 Mar 2010 at 1:12

Done.

Original comment by martin.grotzke on 23 Mar 2010 at 1:26

  • Changed state: Fixed
  • Added labels: Milestone-1.3