Race conditions with Expo FS about react-native-logs HOT 4 CLOSED

Yes the problem is that, unlike RNFS, Expo Fs does not expose the 'appendFile' method. So, for every log, i need to read the contents of the file and write it in full, this obviously causes the problem you described.

There are two ways to fix it:

1. make the procedure synchronous (but this would cause a lot of performance problems)

2. Rewrite the module to work with a queue, string all write requests, and execute them one at a time.

This last option, however, requires a lot of work, I hope they will soon add to expo fs a potion to add lines to a file ...

in the coming weeks I will consider what to do.

from react-native-logs.

@alessandro-bottamedi Yeah, it's a tough one.

I looked a little into RNFS and they have this "append to file" implemented in native code. What about we open a feature request with Expo to add this option? Seems like the code already exists from RNFS so perhaps it's not so much work for Expo to add this option.

Otherwise, I also considered the queue option, and that seems like it adds a lot of complexity. I considered adding it in our logs, because then I could also delay sending logs until we'd initialised our logging settings. That would let me store log levels on the filesystem in the phone for example. But, it's complicated and has the potential to introduce horrible, hard to trace bugs.

One thing to note, I've used a few async / Promise queues, and there are some battle tested libraries out there. Maybe using an existing queue library would be achievable without introducing so much extra code into this package, but adding extra dependencies (which bring their own complications of course).

Anyway, thanks for the considered response. I'll be curious to hear what you think makes sense here.

from react-native-logs.

@chmac @alessandro-bottamedi Hello, please tell me how you solved this problem

from react-native-logs.

@chmac With v 3.0.4 i think I was able to create a custom queue for the ExpoFS module to avoid race conditions issues. I could not use existing queue management packages since they all make use of a DB (sql or realm) and are not compatible with expo.
Please try it and let me know if there are any problems.

from react-native-logs.