Skip to content

[3.1] Update Grpc.AspNetCore to 2.24.0 #15245

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

Merged
merged 1 commit into from
Oct 22, 2019
Merged

Conversation

JamesNK
Copy link
Member

@JamesNK JamesNK commented Oct 21, 2019

Description

Update Grpc.AspNetCore reference in gRPC template to 2.24.0.

Customer impact

2.24.0 fixes a number of issues found in the initial gRPC release that have been reported by customers. Release notes: https://github.com/grpc/grpc-dotnet/releases/tag/v2.24.0-pre1

This PR updates the template so new gRPC projects get a version of gRPC with fixes by default.

Regression

No

Risk

Low

@JamesNK JamesNK added this to the 3.1.0-preview2 milestone Oct 21, 2019
@JamesNK JamesNK requested review from Pilchie and JunTaoLuo October 21, 2019 20:12
@JamesNK JamesNK requested a review from dougbu as a code owner October 21, 2019 20:12
@JamesNK JamesNK added the area-grpc Includes: GRPC wire-up, templates label Oct 21, 2019
@@ -236,7 +236,7 @@
<CastleCorePackageVersion>4.2.1</CastleCorePackageVersion>
<FSharpCorePackageVersion>4.2.1</FSharpCorePackageVersion>
<GoogleProtobufPackageVersion>3.8.0</GoogleProtobufPackageVersion>
<GrpcAspNetCorePackageVersion>2.23.2</GrpcAspNetCorePackageVersion>
<GrpcAspNetCorePackageVersion>2.24.0</GrpcAspNetCorePackageVersion>
Copy link
Contributor

Choose a reason for hiding this comment

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

@JamesNK what helps this change meet the 3.1 bar? @Pilchie any objections?

Copy link
Contributor

Choose a reason for hiding this comment

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

More specifically, how do the fixes in https://github.com/grpc/grpc-dotnet/releases/tag/v2.24.0-pre1 affect the gRPC features of this repo?

Copy link
Member Author

Choose a reason for hiding this comment

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

Updated customer impact.

@Pilchie
Copy link
Member

Pilchie commented Oct 22, 2019

I approve for Preview 2. After that, the bar might be too high for 3.1 without fixes that impact a significant number of users.

@JamesNK JamesNK merged commit 9bb9fc3 into release/3.1 Oct 22, 2019
@JamesNK JamesNK deleted the jamesnk/grpc-2.24-3.1 branch October 22, 2019 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-grpc Includes: GRPC wire-up, templates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants