search

dotnet / runtime

.NET is a cross-platform runtime for cloud, mobile, desktop, and IoT apps.
https://docs.microsoft.com/dotnet/core/
MIT License
15.06k stars 4.69k forks source link

Replace deprecated SecTrustGetCertificateAtIndex API in System.Security.Cryptography.Native.Apple #101349

Open akoeplinger opened 5 months ago

akoeplinger commented 5 months ago

When bumping to macOS12/iOS12.2 in https://github.com/dotnet/runtime/pull/101342 we got a new warning about this API being deprecated.

From @filipnavara

The SecTrustGetCertificateAtIndex deprecation would need a new code path to use SecTrustCopyCertificateChain. To make that efficient we would likely need to rewrite the managed -> native interface.

dotnet-policy-service[bot] commented 5 months ago

Tagging subscribers to 'os-ios': @steveisok, @akoeplinger, @kotlarmilos See info in area-owners.md if you want to be subscribed.

dotnet-policy-service[bot] commented 5 months ago

Tagging subscribers to 'os-tvos': @steveisok, @akoeplinger See info in area-owners.md if you want to be subscribed.

dotnet-policy-service[bot] commented 5 months ago

Tagging subscribers to 'os-maccatalyst': @steveisok, @akoeplinger See info in area-owners.md if you want to be subscribed.

dotnet-policy-service[bot] commented 5 months ago

Tagging subscribers to this area: @dotnet/area-system-security, @bartonjs, @vcsjones See info in area-owners.md if you want to be subscribed.

vcsjones commented 5 months ago

As noted in https://github.com/dotnet/runtime/pull/101342#issuecomment-2068031913, I have a head start on deprecation replacements so I can just include this one in the work.

vcsjones commented 2 months ago

Bumping this to 10.0, it's not blocking for 9.0.