-
Notifications
You must be signed in to change notification settings - Fork 281
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
EnclaveDelegate.Crypto GetEnclaveProvider appears to not be thread safe #1444
Comments
thoughts? |
Are there any other places that the dictionary is mutated? if so those will need to be locked as well. Otherwise looks ok to me. |
Changing it to a ConcurrentDictionary seems to fix it. Let me know what you think. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As a result of issue #1422 we are restarting app service every 8 hours to prevent the null enclave session. Occasionally we are getting the following exception when multiple async calls are made at once.
To reproduce (it doesn't occur every time, but if you run program a few times it will occur)
Expected behavior
s_enclaveProviders should be thread safe and not fail in GetEnclaveProvider
Further technical details
Microsoft.Data.SqlClient version: 4.0.0
.NET target: Core 3.1
SQL Server version: Azure SQL
Attestation: AAS
The text was updated successfully, but these errors were encountered: