Skip to content

Mark HttpClient.Send tests uniformly with UnsupportedOSPlatform for Android, iOS#120716

Merged
ManickaP merged 4 commits intodotnet:mainfrom
eriawan:httpclient_send_add_unsupportedos_attributes
Nov 11, 2025
Merged

Mark HttpClient.Send tests uniformly with UnsupportedOSPlatform for Android, iOS#120716
ManickaP merged 4 commits intodotnet:mainfrom
eriawan:httpclient_send_add_unsupportedos_attributes

Conversation

@eriawan
Copy link
Member

@eriawan eriawan commented Oct 14, 2025

Fixes #104716 by marking HttpClient.Send with UnsupportedOSPlatform with values of android, ios over eixisting.

Please review.

cc @wfurt , @ManickaP

Copilot AI review requested due to automatic review settings October 14, 2025 18:39
@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Oct 14, 2025
Copilot AI reviewed Oct 14, 2025
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR addresses issue #104716 by adding platform-specific annotations to mark HttpClient.Send methods as unsupported on Android and iOS platforms, in addition to the existing browser platform restriction.

Key changes:

  • Added UnsupportedOSPlatform attributes for "android" and "ios" to synchronous Send methods
  • Uncommented previously commented-out platform attributes in HttpClientHandler
  • Updated both implementation and reference assembly files for consistency

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
HttpMessageInvoker.cs Added android and ios platform restrictions to the base Send method
HttpClientHandler.AnyMobile.cs Uncommented android and ios platform attributes for the Send method override
HttpClient.cs Added android and ios platform restrictions to all four Send method overloads
System.Net.Http.cs Updated reference assembly to reflect the new platform restrictions
Comments suppressed due to low confidence (1)

src/libraries/System.Net.Http/ref/System.Net.Http.cs:1

  • Corrected malformed attribute name from 'UnsupportedOSPlatformAttributeUnsupportedOSPlatform' to 'UnsupportedOSPlatform'.
// Licensed to the .NET Foundation under one or more agreements.

@dotnet-policy-service
Copy link
Contributor

dotnet-policy-service bot commented Oct 14, 2025

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

@eriawan eriawan mentioned this pull request Oct 14, 2025
Closed
This was referenced Oct 14, 2025
Open
Open
Closed
@Eriawan-on-Avanade Eriawan-on-Avanade mentioned this pull request Oct 15, 2025
Closed
@karelz karelz added this to the 11.0.0 milestone Oct 21, 2025
ManickaP
ManickaP reviewed Oct 21, 2025
View reviewed changes
@karelz karelz changed the title Fixes #104716 marking HttpClient.Send with UnsupportedOSPlatform with values of android, ios over eixisting Fixes #104716 marking HttpClient.Send with UnsupportedOSPlatform with values of android, ios over existing Oct 29, 2025
@karelz karelz changed the title Fixes #104716 marking HttpClient.Send with UnsupportedOSPlatform with values of android, ios over existing Fixes #104716 marking HttpClient.Send with UnsupportedOSPlatform with values of Android, iOS over existing Oct 29, 2025
@karelz karelz changed the title Fixes #104716 marking HttpClient.Send with UnsupportedOSPlatform with values of Android, iOS over existing Mark HttpClient.Send tests with UnsupportedOSPlatform with values of Android, iOS over existing Oct 29, 2025
@karelz karelz added the test-enhancement Improvements of test source code label Oct 29, 2025
@karelz karelz changed the title Mark HttpClient.Send tests with UnsupportedOSPlatform with values of Android, iOS over existing Mark HttpClient.Send tests uniformly with UnsupportedOSPlatform for Android, iOS Oct 29, 2025
eriawan and others added 2 commits November 2, 2025 20:07
@eriawan
mark HttpClient.Send with UnsupportedOSPlatform with values of androi…
c2850d0 
…d, ios over eixisting
@eriawan
Update src/libraries/System.Net.Http/ref/System.Net.Http.cs
70e6bc9 
accept copilot suggestion to correct UnsupportedOSPlatform attribute name

Co-authored-by: Copilot <[email protected]>
@eriawan eriawan force-pushed the httpclient_send_add_unsupportedos_attributes branch from f910d01 to 5066cb7 Compare November 2, 2025 13:07
@eriawan
Copy link
Member Author

eriawan commented Nov 2, 2025

Rebase again to main branch as of now.

@eriawan eriawan force-pushed the httpclient_send_add_unsupportedos_attributes branch from 5066cb7 to 0ebf851 Compare November 2, 2025 15:28
@build-analysis build-analysis bot mentioned this pull request Nov 2, 2025
Open
3 tasks
ManickaP
ManickaP approved these changes Nov 3, 2025
View reviewed changes
Copy link
Member

@ManickaP ManickaP left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@ManickaP
Copy link
Member

ManickaP commented Nov 3, 2025

Seems like the errors are relevant: API compatibility errors between 'ref/net10.0/System.Net.Http.dll' (left) and 'runtimes/win/lib/net10.0/System.Net.Http.dll' etc.

@eriawan
Copy link
Member Author

eriawan commented Nov 3, 2025

Seems like the errors are relevant: API compatibility errors between 'ref/net10.0/System.Net.Http.dll' (left) and 'runtimes/win/lib/net10.0/System.Net.Http.dll' etc.

@ManickaP
I think those errors are not caused by my PR, they are closely related to this issue on dotnet/arcade: dotnet/arcade#7585

especially this comment: dotnet/arcade#7585 (comment)
Now that ApiCompat has been moved to dotnet/sdk. the problem is still there: dotnet/sdk#35542

@ManickaP
Copy link
Member

ManickaP commented Nov 4, 2025

No, you have to add the same set of attributes to src/libraries/System.Net.Http/src/System/Net/Http/HttpClientHandler.cs as well.
This reproduces locally, so you can test it out yourself.

@eriawan
Copy link
Member Author

eriawan commented Nov 8, 2025

No, you have to add the same set of attributes to src/libraries/System.Net.Http/src/System/Net/Http/HttpClientHandler.cs as well. This reproduces locally, so you can test it out yourself.

@ManickaP thanks! you're right. Hopefully the CI should be green now.

@eriawan
Copy link
Member Author

eriawan commented Nov 8, 2025

@ManickaP

I think those failed CI is not related to this PR, it says something about exception of SSL in Azure DevOps task.
Please review thia again, thanks!

This was referenced Nov 9, 2025
Open
Open
Closed
Open
Open
@ManickaP ManickaP mentioned this pull request Nov 11, 2025
Open
@ManickaP
Copy link
Member

ManickaP commented Nov 11, 2025

/ba-g #121518

@ManickaP ManickaP merged commit a571a94 into dotnet:main Nov 11, 2025
68 of 87 checks passed
@dotnet-maestro dotnet-maestro bot mentioned this pull request Nov 12, 2025
Merged
@eriawan
Copy link
Member Author

eriawan commented Nov 15, 2025

Thanks @ManickaP ! 👍

@eriawan eriawan deleted the httpclient_send_add_unsupportedos_attributes branch November 15, 2025 17:10
@github-actions github-actions bot locked and limited conversation to collaborators Dec 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

Copilot code review Copilot Copilot left review comments

@ManickaP ManickaP ManickaP approved these changes

Assignees

@ManickaP ManickaP

Labels

area-System.Net.Http community-contribution Indicates that the PR has been added by a community member test-enhancement Improvements of test source code

Projects

None yet

Milestone

11.0.0

Development

Successfully merging this pull request may close these issues.

HttpClient.Send not supported in MAUI running on Android or iOS causing PlatformNotSupportedException

4 participants

@eriawan @ManickaP @karelz