missing revalidation?

[setharnold]

Hello, it appears a lock is dropped before a potentially long-running operation, and then reacquired before continuing. This is fine. However, it appears that there is no confirmation that the update is still legal after reacquiring the lock.

Are there any operations that could be performed while fetch() is executing that would invalidate the cache in such a way that the insert should be failed instead of continuing?

utils/cache/cache.go

Line 126 in bf9cc5b

// Fetch the data without the mutex held

Thanks