Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

don't working anymore with chrome when using dependency write_file_atomic@1.3.4 #61

Closed
miouge opened this issue May 5, 2017 · 5 comments

Comments

@miouge
Copy link

miouge commented May 5, 2017

Hello
can't explain really why but my cloud app was working
using session-file-store@1.0.0 using as dependency write_file_atomic@1.3.1
and is not working anymore
using session-file-store@1.0.0 using as dependency write_file_atomic@1.3.4

the problem appear only when using chrome but not firefox (very strange, but very reproductible)
symptoms are that passport object are not deserialized into req.user.

just uninstalling the write_file_atomic@1.3.4 and install the write_file_atomic@1.3.1 package fix the problem
our nodeJs server on linux used also for authentification
passport@0.3.2
passport-local@1.0.0
express@4.15.2
express-session@1.15.1
hope this can help others.

I beg your pardon for my poor language level (i'm french)

Philippe

@DonMartin76
Copy link

Hello @miouge, this greatly helped me. All of a sudden, I got this really strange misbehaviour that sessions did not seem to be persisted, or, in some cases, later (not reproducible). It did not even seem tied to a browser, but probably rather to a specific runtime, such as Docker or non-Docker.

Pinning write-file-atomic to 1.3.1 in my package.json helped me, as npm then re-uses that version for session-file-store. See Haufe-Lexware/wicked.haufe.io#72. I will file an issue with that package as well.

@valery-barysok
Copy link
Owner

lock write-file-atomic version to 1.3.1

@iarna
Copy link

iarna commented Aug 18, 2017

I encourage you all to try write-file-atomic@2.3.0 which serializes access to the same file. (The breaking change in 2.0.0 was dropping support for 0.10 and 0.12.)

@T3chArmy
Copy link

Forcing write-file-atomic to v2.3.0 fixed it for me!

@valery-barysok
Copy link
Owner

@T3chArmy, great! will update dependency in near future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants