Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

spanner: requiring @google-cloud/spanner stops the process from exiting on SIGTERM/SIGINT

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#9,513 0 bình luận 0 reaction 1 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@alkatrivedi đang làm issue này rồi.

Từ ngày 13/10/2026.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
78/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
nodejs, typescript
Lĩnh vực
backend

Hướng nghiên cứu

Bắt đầu với handwritten/spanner/src/index.ts, nơi các listener SIGINT và SIGTERM được đăng ký, rồi chạy bản tái hiện tối thiểu được liên kết bằng npm install && npm test. Xác nhận hành vi của tiến trình khi có và không có @google-cloud/spanner; được xem là hoàn tất khi việc tải thư viện không còn ngăn cản hành vi tắt tiến trình do tín hiệu như mong đợi.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

api: spanner
Please make sure you have searched for information in the following guides.
Library Name

@google-cloud/spanner

A screenshot that you have tested with "Try this API".

Not API related

Link to the code that reproduces this issue. A link to a public Github Repository or gist with a minimal reproduction.

https://gist.github.com/wildan2711/1630361a07bd531792be280519d8e3fa

A step-by-step description of how to reproduce the issue, based on the linked reproduction.

Runnable reproduction: https://gist.github.com/wildan2711/1630361a07bd531792be280519d8e3fa (npm install && npm test)

Minimal version:

// repro.js
require('@google-cloud/spanner');
console.log('SIGTERM listeners:', process.listenerCount('SIGTERM'));
setInterval(() => {}, 1000); // anything that keeps the event loop alive, e.g. an HTTP server

node repro.js &
kill -TERM $!   # process keeps running
kill -INT  $!   # still running; only SIGKILL stops it

Remove the require line and the same script exits on SIGTERM as expected.

A clear and concise description of what the bug is, and what you expected to happen.

Requiring @google-cloud/spanner makes a long-running Node.js process ignore SIGTERM and SIGINT. When the module loads, it registers process.on('SIGINT') and process.on('SIGTERM') listeners in handwritten/spanner/src/index.ts that run the metrics cleanup() but never exit or re-raise the signal. The process then keeps running until it gets SIGKILL. This applies even if no Spanner client is ever created. In practice, Cloud Run, GKE and App Engine instances run until the end of the shutdown grace period instead of stopping, docker stop and process managers leave orphaned processes holding their ports, and Ctrl+C doesn't stop a dev server.

Expected: loading the library doesn't change how the process responds to signals. A process that would exit on SIGTERM or SIGINT without @google-cloud/spanner should still exit with it.

A clear and concise description WHY you expect this behavior, i.e., was it a recent change, there is documentation that points to this behavior, etc. **
  • It's a regression. 8.0.0 registers no signal listeners and the process exits normally. The listeners first appear in 8.1.0 and are still present in 9.0.0 and on main. We noticed when upgrading from 6.x to 8.6.0: our services stopped shutting down on SIGTERM.
  • Node.js documents this effect. Per the signal events docs, "SIGTERM and SIGINT have default handlers on non-Windows platforms that reset the terminal mode before exiting with code 128 + signal number. If one of these signals has a listener installed, its default behavior will be removed (Node.js will no longer exit)." So any listener that doesn't exit stops the process from shutting down.
  • The code's own comment says the process should exit. It reads // For signals (let process exit naturally), which shows the listeners were meant to flush metrics without blocking shutdown. But a process with an open server, socket or timer never exits "naturally", so in practice they block it.
  • Libraries generally shouldn't own process signals. Shutdown on SIGTERM and SIGINT belongs to the application or its framework, which may need its own graceful-shutdown logic. A dependency that silently disables the default makes that impossible to see, and the only workaround is removing the listeners by hand.
Ngôn ngữ chính
TypeScript
Star
3.2k
Fork
724
Merge trung bình
1 ngày 23 giờ
Pull request đã merge (30 ngày)
141

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của googleapis/google-cloud-node

Tất cả issue của googleapis/google-cloud-node

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.