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/6.0-preview5] Remove System.Security.Cryptography.Cng from ASP.NET transport package #54456

Conversation

github-actions[bot]
Copy link
Contributor

Backport of #54428 to release/6.0-preview5

/cc @ericstj

Customer Impact

Testing

Risk

dotnet-maestro bot and others added 13 commits May 20, 2021 10:06
….2 (#52933)

Microsoft.NETCore.Runtime.ICU.Transport
 From Version 6.0.0-preview.5.21267.1 -> To Version 6.0.0-preview.5.21268.2

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Tarek Mahmoud Sayed <tarekms@microsoft.com>
Co-authored-by: Jo Shields <joshield@microsoft.com>
…tocol optional (#53072)

* Make the lookup for getApplicationProtocol optional

The method getApplicationProtocol in javax.net.ssl.SSLEngine is only supported on API level 29 and above, so running on older devices would result in a crash. This change makes the initial method lookup optional and AndroidCryptoNative_SSLStreamGetApplicationProtocol in pal_sslstream.c will error if it is not supported.

Fixes #52965

* Feedback

Co-authored-by: Steve Pfister <steve.pfister@microsoft.com>
…26.5 (#53310)

Microsoft.NET.Runtime.Emscripten.2.0.12.Node.win-x64
 From Version 6.0.0-preview.5.21267.1 -> To Version 6.0.0-preview.5.21276.5

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
…ion logic (#53337)

* Add JSON source-gen mode that emits serialization logic

* Fix System.Net.Http.Json test issues

* Fix System.Text.Json test issues

* Make check to determine if fast-path can be used more efficient

* Address review feedback

* Improve derived-JsonSerializerContext detection and support

* Address review feedback; reenable tests, and simplify object metadata init

* Fix formatting

Co-authored-by: Layomi Akinrinade <layomia@gmail.com>
Co-authored-by: Layomi Akinrinade <laakinri@microsoft.com>
* Update dependencies from https://github.com/dotnet/emsdk build 20210527.1

Microsoft.NET.Runtime.Emscripten.2.0.12.Node.win-x64
 From Version 6.0.0-preview.5.21276.5 -> To Version 6.0.0-preview.5.21277.1

* Update dependencies from https://github.com/dotnet/emsdk build 20210527.3

Microsoft.NET.Runtime.Emscripten.2.0.12.Node.win-x64
 From Version 6.0.0-preview.5.21276.5 -> To Version 6.0.0-preview.5.21277.3

* Update dependencies from https://github.com/dotnet/emsdk build 20210528.1

Microsoft.NET.Runtime.Emscripten.2.0.12.Node.win-x64
 From Version 6.0.0-preview.5.21276.5 -> To Version 6.0.0-preview.5.21278.1

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
* Update dependencies from https://github.com/dotnet/emsdk build 20210528.3

Microsoft.NET.Runtime.Emscripten.2.0.12.Node.win-x64
 From Version 6.0.0-preview.5.21278.1 -> To Version 6.0.0-preview.5.21278.3

* Update dependencies from https://github.com/dotnet/emsdk build 20210528.4

Microsoft.NET.Runtime.Emscripten.2.0.12.Node.win-x64
 From Version 6.0.0-preview.5.21278.1 -> To Version 6.0.0-preview.5.21278.4

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
…8.1 (#53443)

Microsoft.NET.ILLink.Tasks
 From Version 6.0.100-preview.5.21267.3 -> To Version 6.0.100-preview.5.21278.1

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
…ack in the Mono workload (#53426)

* Remove unnecessary reference to iOS workload pack in the Mono workload

* Fix tvOS x64 cross reference

Co-authored-by: Steve Pfister <steve.pfister@microsoft.com>
Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
…arm platforms (#52942)

Co-authored-by: Aleksey Kliger <alklig@microsoft.com>
@ghost
Copy link

ghost commented Jun 19, 2021

Tagging subscribers to this area: @bartonjs, @vcsjones, @krwq, @GrabYourPitchforks
See info in area-owners.md if you want to be subscribed.

Issue Details

Backport of #54428 to release/6.0-preview5

/cc @ericstj

Customer Impact

Testing

Risk

Author: github-actions[bot]
Assignees: -
Labels:

area-System.Security

Milestone: -

@vcsjones
Copy link
Member

Preview 5 already shipped. Shouldn't this target release/6.0-preview6?

@ViktorHofer ViktorHofer changed the base branch from release/6.0-preview5 to release/6.0-preview6 June 21, 2021 12:11
@ViktorHofer
Copy link
Member

@ericstj I will retrigger the backport action to create a new PR as retargeting to P6 didn't work.

@ViktorHofer ViktorHofer deleted the backport/pr-54428-to-release/6.0-preview5 branch June 21, 2021 12:12
@ericstj
Copy link
Member

ericstj commented Jun 21, 2021

Ugh, my mistake here, apologies for this. Was submitting from a phone :(

@ghost ghost locked as resolved and limited conversation to collaborators Jul 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants