search

amplitude / experiment-ios-client

MIT License
2 stars 9 forks source link

fix: potential crash with concurrent storage map copy and write #13

Closed bgiori closed 2 years ago

bgiori commented 2 years ago

Summary

Storage.getAll() copies the storage cache dictionary which is not thread safe with regards to writing to the map concurrently.

This change synchronizes getAll() with the other storage access in storeVariants()

Checklist

github-actions[bot] commented 2 years ago

:tada: This PR is included in version 1.4.1 :tada:

The release is available on:

Your semantic-release bot :package::rocket: