-
Notifications
You must be signed in to change notification settings - Fork 42
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
Creating new files is not multi-goroutine safe #40
Comments
@sfc-gh-thardie Yep, this is an issue that unfortunately ends up affecting From an API perspective, we could do something similar to the approach on the recent |
I'm interested in this as well |
Anyone interested on working on this feel free to propose a PR against the branch tracking for v6. |
If you call
New()
on thestorage
objects from multiple goroutines at the same time, go runtime may error withfatal error: concurrent map writes
. This happens because thefiles
map inside thestorage
struct has no concurrency protection around multiple access.Other concurrency issues probably exist for accessing the
children
map inside thestorage
object.The text was updated successfully, but these errors were encountered: