kleydon / prisma-session-store

Express session store for Prisma
MIT License
116 stars 18 forks source link

Workaround for Issue #88 #104

Closed kleydon closed 1 year ago

kleydon commented 1 year ago

Concurrent invocations of touch() or set() for the same session id can result in problems. (See isssue 88).

Practically, this may happen when a browser concurrently loads multiple resources for the same site/page.

The problem may be limited to projects backed by an SQLite db, but this has not yet been carefully tested; prisma or express-session may alternately / additionally be implicated.

As a work-around for now, this PR dis-allows concurrent calls to touch() or set() for the same session id by default - but provides options to over-ride this new behavior, in case there happen to be scenarios for which the "fix" proves more problematic than helpful.

kleydon commented 1 year ago

:tada: This PR is included in version 3.1.10 :tada:

The release is available on:

Your semantic-release bot :package::rocket: