forked from dotnet/android
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Bump to dotnet/installer@5c2d8ef 6.0.100-rc.2.21418.44 (dotnet#6204)
Changes: dotnet/installer@78a1bc3...5c2d8ef Changes: dotnet/linker@5b2391c...5b2391c Changes: dotnet/runtime@14b34eb...5a5d7f0 Context: https://github.com/dotnet/runtime/blob/0f5b75344d5858d76da403735ee34c71d9d69d54/src/libraries/System.Net.Http/src/System/Net/Http/HttpClientHandler.AnyMobile.cs#L776 Context: dotnet/runtime@57dd919 Context: dotnet/runtime#56089 Context: dotnet/runtime#57800 Updates: * Microsoft.Dotnet.Sdk.Internal: from 6.0.100-rc.1.21418.8 to 6.0.100-rc.2.21418.44 * Microsoft.NET.ILLink.Tasks: from 6.0.100-preview.6.21416.1 to 6.0.100-preview.6.21418.3 * Microsoft.NETCore.App.Ref: from 6.0.0-rc.1.21417.1 to 6.0.0-rc.2.21417.16 Update `Xamarin.Android.Net.AndroidMessageHandler` so that additional properties are specified. Commit dotnet/runtime@57dd919a updated `HttpMessageHandler` so that additional properties were no longer "unsupported" on Android, meaning they needed to exist. As they didn't yet exist, the linker emitted warnings: ILLink : warning IL2037: System.Net.Http.HttpClientHandler.GetClientCertificateOptions(): No members were resolved for 'get_ClientCertificateOptions'. ILLink : warning IL2037: System.Net.Http.HttpClientHandler.GetClientCertificates(): No members were resolved for 'get_ClientCertificates'. ILLink : warning IL2037: System.Net.Http.HttpClientHandler.SetClientCertificateOptions(ClientCertificateOption): No members were resolved for 'set_ClientCertificateOptions'. We need to add the missing members that are now expected: partial class AndroidMessageHandler { public ClientCertificateOption ClientCertificateOptions { get; set; } public X509CertificateCollection ClientCertificates { get; set; } // … } Note that these new properties *must* be public, as `Type.GetMethod(string)` is used, which only finds public members. We can also update `AndroidClientHandlerTests.cs` and remove the `catch(TargetInvocationException)` blocks added in 1e5bfa3, as dotnet/runtime#56089 has been fixed. Begin ignoring `AotTests.NoSymbolsArgShouldReduceAppSize()`, as it started failing. See dotnet/runtime#57800. Finally, `SslProtocols.Tls13` is only valid on API-29+. Update `AndroidMessageHandler.SslProtocols` so that `SslProtocols.Tls13` is only used on appropriate platforms. Co-authored-by: Jonathan Peppers <jonathan.peppers@microsoft.com>
- Loading branch information
1 parent
d304060
commit 1dbb547
Showing
5 changed files
with
38 additions
and
21 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters