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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add proper signal handling for graceful shutdowns #30

Merged
merged 1 commit into from
Jan 16, 2025

Conversation

mikesir87
Copy link
Member

Added signal handlers to the main script file to shutdown the app when SIGINT and SIGTERM signals are sent. Otherwise, it hangs when trying to Ctrl+C a running container.

Copy link

Your image dockerdevrel/catalog-service-node:pr-30 critical: 0 high: 0 medium: 1 low: 24
Current base image node:22-bookworm-slim critical: 0 high: 0 medium: 0 low: 24
Policy Status
policy status (4/7 policies met, 2 missing data)
Status Policy Results
Default non-root user
No AGPL v3 licenses 0 packages
No fixable critical or high vulnerabilities critical: 0 high: 0 medium: 0 low: 0
No high-profile vulnerabilities critical: 0 high: 0 medium: 0 low: 0
No outdated base images No data
Learn more ↗
No unapproved base images No data
⚠️ Missing supply chain attestation(s) 2 deviations

Copy link

Overview

Image reference dockerdevrel/catalog-service-node:latest dockerdevrel/catalog-service-node:pr-30
- digest b630c97cef9b 62a8e884511d
- tag latest pr-30
- environment production
- provenance 781663d 1d3c2c4
- vulnerabilities critical: 0 high: 0 medium: 1 low: 24 critical: 0 high: 0 medium: 1 low: 24
- platform linux/amd64 linux/amd64
- size 84 MB 84 MB (+92 B)
- packages 330 330
Base Image node:22-bookworm-slim
also known as:
22-slim
22.13-bookworm-slim
22.13-slim
22.13.0-bookworm-slim
22.13.0-slim
jod-bookworm-slim
jod-slim
lts-bookworm-slim
lts-slim
node:22-bookworm-slim
also known as:
22-slim
22.13-bookworm-slim
22.13-slim
22.13.0-bookworm-slim
22.13.0-slim
jod-bookworm-slim
jod-slim
lts-bookworm-slim
lts-slim
- vulnerabilities critical: 0 high: 0 medium: 0 low: 24 critical: 0 high: 0 medium: 0 low: 24
Labels (3 changes)
  • ± 3 changed
  • 5 unchanged
-org.opencontainers.image.created=2025-01-15T22:47:28.805Z
+org.opencontainers.image.created=2025-01-16T04:18:39.997Z
 org.opencontainers.image.description=
 org.opencontainers.image.licenses=CC0-1.0
-org.opencontainers.image.revision=781663d8eeebad7825a85796e942c08bc774cbd0
+org.opencontainers.image.revision=1d3c2c4f876c8373f2589138924edf303ae40768
 org.opencontainers.image.source=https://github.com/dockersamples/catalog-service-node
 org.opencontainers.image.title=catalog-service-node
 org.opencontainers.image.url=https://github.com/dockersamples/catalog-service-node
-org.opencontainers.image.version=v0.2.0
+org.opencontainers.image.version=pr-30
Policies (0 improved, 1 worsened, 2 missing data)
Policy Name dockerdevrel/catalog-service-node:latest dockerdevrel/catalog-service-node:pr-30 Change Standing
Default non-root user No Change
No AGPL v3 licenses No Change
No fixable critical or high vulnerabilities No Change
No high-profile vulnerabilities No Change
No outdated base images ❓ No data
No unapproved base images ❓ No data
Supply chain attestations ⚠️ 2 +2 Worsened

@mikesir87 mikesir87 merged commit fa7565c into main Jan 16, 2025
18 checks passed
@mikesir87 mikesir87 deleted the add-signal-handling branch January 16, 2025 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant