Don't expose TrustedCertificatesDirectory() and StartNewTlsSessionContext() to NetFx

[steveharter]

Fixes #113947

Will need to backport to v9 and v8.

[ghost]

Note regarding the new-api-needs-documentation label:

This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, please make sure the API implementation in the src *.cs file is documented with triple slash comments, so the PR reviewers can sign off that change.

[ghost]

Note regarding the new-api-needs-documentation label:

This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, please make sure the API implementation in the src *.cs file is documented with triple slash comments, so the PR reviewers can sign off that change.

[Copilot]

Pull Request Overview

This PR prevents exposing the TrustedCertificatesDirectory property and StartNewTlsSessionContext method to NetFx by conditionally compiling them for NET targets only.

• Guard TrustedCertificatesDirectory with #if NET and mark it with the UnsupportedOSPlatformAttribute.
• Guard StartNewTlsSessionContext with #if NET and mark it accordingly.

[dotnet-policy-service]

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

[teo-tsirpanis]

That's a breaking change, isn't it?

[steveharter]

That's a breaking change, isn't it?

No. The method was just introduced and is not implemented on .NET Framework.

[ericstj]

Change looks OK.

It would be nice if we could make API Compat add an additional compatibility tuple for these cases where we don't build for NETFX but expect to be compatible with an assembly in the targeting pack @ViktorHofer

[ViktorHofer]

If you refer to following placeholder files in a comparison tuple, yes we should support that. This is tracked under dotnet/sdk#18165 and dotnet/sdk#27822 attempted to fix that. Never found time to finish it.

[steveharter]

/backport to release/9.0-staging

[github-actions]

Started backporting to release/9.0-staging: https://github.com/dotnet/runtime/actions/runs/14644505815

[steveharter]

/backport to release/8.0-staging

[github-actions]

Started backporting to release/8.0-staging: https://github.com/dotnet/runtime/actions/runs/14648563757