Skip to content
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

[release/5.0-rc2] Backport Browser Unsupported Attribute Changes Properly #42071

Conversation

mdh1418
Copy link
Member

@mdh1418 mdh1418 commented Sep 10, 2020

Fixes Issue
N/A

Master PR
#41094 #41184 #41271 #41694 #41683 #41682 #41700 #40924 #41891 #41963 #41985

Description

The Platform Compatibility Analyzer can be used to identify APIs that are problematic on other specific platforms or deprecated. With the attribute UnsupportedOSPlatformAttribute("browser"), numerous APIs that are not supported on Browser WASM are marked for the platform compatibility analyzer to pick up.

Customer Impact

Without this fix, customers will not be warned against using certain unsupported APIs on Browser WASM.

Risk

Low.

@mdh1418 mdh1418 added arch-wasm WebAssembly architecture area-Meta labels Sep 10, 2020
@mdh1418 mdh1418 added the Servicing-consider Issue for next servicing release review label Sep 10, 2020
@marek-safar
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@mdh1418 mdh1418 force-pushed the mdhwang/backport_unsupported_browser_attributes_properly branch from eaafb86 to 42e4239 Compare September 10, 2020 14:18
@leecow leecow added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Sep 10, 2020
@Anipik Anipik changed the base branch from release/5.0 to release/5.0-rc2 September 10, 2020 18:50
@Anipik Anipik changed the title [release/5.0] Backport Browser Unsupported Attribute Changes Properly [release/5.0-rc2] Backport Browser Unsupported Attribute Changes Properly Sep 10, 2020
@marek-safar
Copy link
Contributor

@mdh1418 could you please work on resolving the conflict

mdh1418 and others added 11 commits September 11, 2020 09:47
…net#41094)

* [wasm] System.ComponentModel enable platform attributes

* [wasm] Mark ExtendedProtectionPolicyTypeConverter.ConvertTo as unsupported

* [wasm] Mark System.ComponentModel.TypeDescriptor.CreateInstance as unsupported

* [wasm] Mark System.ComponentModel.TypeDescriptionProvider.CreateInstance as unsupported

* [wasm] Mark System.ComponentModel.LicenseManager.CreateWithContext as unsupported

* [wasm] Mark System.ComponentModel.MaskedTextProvider.Clone as unsupported

Co-authored-by: Mitchell Hwang <mitchell.hwang@microsoft.com>
* [wasm] Mark System.Console APIs as unsupported on Browser

* System.Console remove unsupported attribute from Console.Clear()

* System.Console mark SetIn as unsupported on Browser

* System.Console move UnsupportedOSPlatform attributes down to the accessor

Co-authored-by: Mitchell Hwang <mitchell.hwang@microsoft.com>
…tnet#41271)

Co-authored-by: Mitchell Hwang <mitchell.hwang@microsoft.com>
…#41694)

Co-authored-by: Mitchell Hwang <mitchell.hwang@microsoft.com>
…net#41683)

* [wasm] System.IO.Compression.Brotli enable platform attributes

* Mark System.IO.Compression.Brotli unsupported at assembly level

Co-authored-by: Mitchell Hwang <mitchell.hwang@microsoft.com>
…er (dotnet#41682)

* [wasm] System.IO.FileSystem.Watcher enable platform attributes

* Mark System.IO.FileSystem.Watcher unsupported at assembly level

Co-authored-by: Mitchell Hwang <mitchell.hwang@microsoft.com>
…otnet#41700)

* [wasm] System.IO.IsolatedStorage enable platform attributes

* Mark System.IO.IsolatedStorage Unsupported at assembly level

Co-authored-by: Mitchell Hwang <mitchell.hwang@microsoft.com>
…t#41891)

* System.Threading.ThreadPool enable platform attributes

* Mark System.Threading.ThreadPool APIs unsupported on browser

* System.Threading.ThreadPool Add Unsupported attribute to other ThreadPool files

* Remove Unsupported attributes from BindHandle

* Add windows Supported Attribute to BindHandle

Co-authored-by: Mitchell Hwang <mitchell.hwang@microsoft.com>
… on Browser (dotnet#41963)

* Mark System.Net.WebSockets.ClientWebSocketOptions APIs as unsupported on Browser

* Add the attributes to non-browser version of ClientWebSocketOptions class to avoid build error

* Add using

* Include platform attributes
…otnet#41985)

Co-authored-by: Mitchell Hwang <mitchell.hwang@microsoft.com>
@mdh1418 mdh1418 force-pushed the mdhwang/backport_unsupported_browser_attributes_properly branch from 42e4239 to d31a893 Compare September 11, 2020 13:50
@marek-safar marek-safar merged commit 320a99e into dotnet:release/5.0-rc2 Sep 11, 2020
@mdh1418 mdh1418 deleted the mdhwang/backport_unsupported_browser_attributes_properly branch September 15, 2020 19:27
@ghost ghost locked as resolved and limited conversation to collaborators Dec 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
arch-wasm WebAssembly architecture area-Meta Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants