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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't set ignore status for SIGTERM and SIGINT on Linux #3174

Merged
merged 1 commit into from
Apr 18, 2024
Merged

Conversation

gwynne
Copy link
Member

@gwynne gwynne commented Apr 17, 2024

These changes are now available in 4.92.7

Changes the behavior of ServeCommand's signal handling setup to more closely match that of swift-service-lifecycle. Hopefully finally solves #2502 馃

@gwynne gwynne added bug Something isn't working semver-patch Internal changes only labels Apr 17, 2024
@gwynne gwynne requested review from 0xTim, MahdiBM and ptoffy April 17, 2024 02:12
@gwynne gwynne merged commit 096c519 into main Apr 18, 2024
16 checks passed
@gwynne gwynne deleted the gwynne-patch-1 branch April 18, 2024 03:39
keniwhat pushed a commit to keniwhat/vapor that referenced this pull request Apr 27, 2024
* main:
  Patch configuration and log actual port on startup (vapor#3160)
  Update provider tests to 5.10 (vapor#3178)
  Migrate to Async NIOFileIO APIs (vapor#3167)
  Removed streamFile deprecation + deactivated advancedETagComparison by default (vapor#3177)
  Remove HeadResponder (vapor#3147)
  Advanced ETag Comparison now supported (vapor#3015)
  Enabled Request Decompression By Default (vapor#3175)
  HTTP2 Response Compression/Request Decompression (vapor#3126)
  Don't set ignore status for SIGTERM and SIGINT on Linux (vapor#3174)
  Fix typos across the codebase (vapor#3162)
  Fix some Sendable warnings on 5.10 (vapor#3158)
  Allow `HTTPServer`'s configuration to be dynamically updatable (vapor#3132)
  Fix issue when client disconnects midway through a stream (vapor#3102)
  Fix handling of "flag" URL query params (vapor#3151)
  Bump the dependencies group with 1 update (vapor#3148)
  Merge Async Tests (vapor#3141)
  Fix URI handling with multiple slashes and variable components. (vapor#3143)
  Fix broken URI behaviors (vapor#3140)

# Conflicts:
#	Package.swift
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working semver-patch Internal changes only
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants