Skip to content

Handle header with errors and endStream = true (1.57.x backport)#10412

Merged
larry-safran merged 1 commit intogrpc:masterfrom
larry-safran:backport-is_10364-1.57
Jul 24, 2023
Merged

Handle header with errors and endStream = true (1.57.x backport)#10412
larry-safran merged 1 commit intogrpc:masterfrom
larry-safran:backport-is_10364-1.57

Conversation

@larry-safran
Copy link
Contributor

  • Eliminate NPE by skipping further processing when stream is defined, but doesn't have a property for streamKey (header processing identified an error)

Backport of #10384

* Eliminate NPE by skipping further processing when stream is defined, but doesn't have a property for streamKey (header processing identified an error)

Fixes grpc#10364

* Add unit test for missing content type
@larry-safran larry-safran requested a review from YifeiZhuang July 24, 2023 20:24
@larry-safran larry-safran merged commit 08a44b9 into grpc:master Jul 24, 2023
@larry-safran larry-safran deleted the backport-is_10364-1.57 branch July 24, 2023 21:33
@larry-safran larry-safran restored the backport-is_10364-1.57 branch July 24, 2023 21:43
@larry-safran larry-safran deleted the backport-is_10364-1.57 branch July 28, 2023 22:00
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants