`Change graceful_shutdown function behavior` PR can cause tonic servers to hang `serve_with_incoming_shutdown` in two different cases

オープン

#4,170 2026年8月27日に作成コメント 4 件リアクション 0 件担当者 1 名 GitHub で見る

@seanmonstar がすでに取り組んでいます。

2026年8月27日から。

評価

この issue はまだ評価されていません。

説明

A-http2 C-bug

Version

Foudn in hyper 1.5.1 and later

Platform

25.5.0 (macOS), though it is also applying on our linux bots

Summary

At OneSignal we have a number of tonic gRPC services we maintain. After a recent update of dependencies on one of them, we noticed that it "randomly" would hang during shutdown for an hour or more.

After a week of chasing the wild goose, we have isolated it to a single PR introduced in hyper 1.5.1:

https://github.com/hyperium/hyper/pull/3729

In these cases a tonic server:

            Server::builder()
                .add_service(GreeterServer::new(MyGreeter::default()))
                .serve_with_incoming_shutdown(incoming, async move {
                    shutdown_grpc_token.cancelled().await
                })
                .await
                .unwrap();

would never return and block for an hour or more after the shutdown_grpc_token CancellationToken was canceled.

We have a primary example non_upgrade_connections_can_block_shutdown and a secondary one we found along the way client_not_responding_example in this self contained example:

https://github.com/chamons/hyper_hung_client_example

With a patch.crates-io line, we can resolve this branch with the PR reverted and both of these example begin acting as expected.

Expected Behavior

It is not expected that passive tonic gRPC connections to a server that are unused would block shutdown. Before 1.5.1 they did not.

Actual Behavior

We block until the upstream instance restarts or otherwise drops all of its connections.

Additional Context

No response

主要言語: Rust
スター: 16.3k
フォーク: 1.8k
平均マージ: 1日 23時間
マージ済み PR（30日）: 13

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
リポジトリをフォークし、ブランチを切って変更します。
issue 番号を参照したプルリクエストを送ります。

hyperium/hyper のほかの issue

Publicly reexport the http crate オープン

C-feature

難易度 1/5 1時間未満初心者へのやさしさ 65/100

hyperium/hyper#2652 · リアクション 4 件 · 2021年9月18日に作成
Allow connection pools to stop reusing an HTTP/2 connection before its keep-alive timeout オープン

C-feature

難易度 5/5 1週間以上初心者へのやさしさ 35/100

hyperium/hyper#4186 · コメント 1 件 · 2026年9月10日に作成
HTTP/1 client connection is never closed or pooled when a response completes while the request body is still unsent ((Reading::KeepAlive, Writing::Body) is a terminal state) オープン

C-bug S-waiting-on-author

hyperium/hyper#4176 · コメント 4 件 · 担当者 1 名 · 2026年8月28日に作成
Body dropped too late when an h1 handshake hits an error オープン

C-bug

難易度 4/5 3〜5日初心者へのやさしさ 48/100

hyperium/hyper#4122 · コメント 5 件 · 2026年7月2日に作成
HTTP CONNECT proxy tunnel has lax response parsing and loses early data オープン

A-client C-bug E-pr-welcome K-hyper-util

難易度 4/5 3〜5日初心者へのやさしさ 48/100

hyperium/hyper#4095 · コメント 7 件 · 2026年6月3日に作成

hyperium/hyper の issue をすべて見る

似ている issue

[coverage-improver] Cover empty-artifact fallback root in build_plugin_plans オープン

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

TheLarkInn/aipm#2413 · 2026年9月24日に作成
AGENTS.md points to src/settings_window.rs, src/config.rs, and src/app.rs, which were split into directories オープン

documentation

難易度 1/5 1時間未満初心者へのやさしさ 90/100

alexgorbatchev/simple-ptt#15 · 2026年9月24日に作成
bench: run.sh's skip-update message always cites bench/skips.txt オープン

tooling

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

svandragt/vivace#313 · 2026年9月24日に作成
`t/vm/frames/light-call-frame-writes-dedup.t` wall-clock ratio check fails under `-j4` load オープン

todo:ticket

難易度 2/5 1〜3時間初心者へのやさしさ 70/100

tokuhirom/mutsu#9299 · 2026年9月24日に作成
fix(taiko-client-rs): prevent BeaconClient panic on zero beacon spec values オープン

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

taikoxyz/taiko-mono#22168 · コメント 1 件 · 2026年9月24日に作成

Rust の issue をもっと見る