Guard against race conditions in NdrError.log - Githubissues

NHSDigital / ndr_error

Error logging Rails engine

MIT License

1 stars 3 forks source link

Guard against race conditions in NdrError.log #22

Open joshpencheon opened 4 years ago

joshpencheon commented 4 years ago

If a multi-process application is under particularly high traffic, it is plausible that multiple instances have the same probably contemporaneously, and race conditions could be hit:

creating fingerprints by digest
storing logs up to limit
updating fingerprint time/count occurrence details

We should review these cases, to be more defensive.