You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Only modification here is that I've wrapped fetchImage in Swift concurrency, but that doesn't change the underly safety issue.
Fundamentally, the jsonDecoderBuilder property is unsafe because it's mutable yet accessible from multiple threads.
There are multiple possible solutions here but the simplest is likely a lock wrapper to provide safe access. I don't see such a construct in the project already, so I'm just reaching out to see if there's another preferred solution before I try to get the project fully building and testable locally.
The text was updated successfully, but these errors were encountered:
Running with the thread sanitizer, occasionally multiple requests will trigger a thread sanitizer warning. For example:
Only modification here is that I've wrapped
fetchImage
in Swift concurrency, but that doesn't change the underly safety issue.Fundamentally, the
jsonDecoderBuilder
property is unsafe because it's mutable yet accessible from multiple threads.There are multiple possible solutions here but the simplest is likely a lock wrapper to provide safe access. I don't see such a construct in the project already, so I'm just reaching out to see if there's another preferred solution before I try to get the project fully building and testable locally.
The text was updated successfully, but these errors were encountered: