Skip to content

Chore: add signal-handling to run-blocking#2134

Merged
nichtsfrei merged 1 commit intomainfrom
signal-handling
Mar 11, 2026
Merged

Chore: add signal-handling to run-blocking#2134
nichtsfrei merged 1 commit intomainfrom
signal-handling

Conversation

@nichtsfrei
Copy link
Member

@nichtsfrei nichtsfrei commented Feb 13, 2026

Adds rudimentary signal handling to openvasd.

SC-1521

@github-actions
Copy link

github-actions bot commented Feb 13, 2026

Dependency Review

The following issues were found:
  • ✅ 0 vulnerable package(s)
  • ✅ 0 package(s) with incompatible licenses
  • ✅ 0 package(s) with invalid SPDX license definitions
  • ⚠️ 2 package(s) with unknown licenses.
See the Details below.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA b23aae3.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

License Issues

rust/crates/greenbone-scanner-framework/Cargo.toml

PackageVersionLicenseIssue Type
signal-hook>= 0.4.3, < 0.5.0NullUnknown License
signal-hook-tokio>= 0.4.0, < 0.5.0NullUnknown License
Allowed Licenses: 0BSD, AGPL-3.0-or-later, Apache-2.0, BlueOak-1.0.0, BSD-2-Clause, BSD-3-Clause-Clear, BSD-3-Clause, BSL-1.0, bzip2-1.0.6, CAL-1.0, CC-BY-3.0, CC-BY-4.0, CC-BY-SA-4.0, CC0-1.0, EPL-2.0, GPL-1.0-or-later, GPL-2.0-only, GPL-2.0-or-later, GPL-2.0, GPL-3.0-only, GPL-3.0-or-later, GPL-3.0, ISC, LGPL-2.0-only, LGPL-2.0-or-later, LGPL-2.1-only, LGPL-2.1-or-later, LGPL-2.1, LGPL-3.0-only, LGPL-3.0, LGPL-3.0-or-later, MIT, MIT-CMU, MPL-1.1, MPL-2.0, OFL-1.1, PSF-2.0, Python-2.0, Python-2.0.1, Unicode-3.0, Unicode-DFS-2016, Unlicense, Zlib, ZPL-2.1

OpenSSF Scorecard

PackageVersionScoreDetails
cargo/signal-hook 0.4.3 🟢 4
Details
CheckScoreReason
Maintained🟢 1021 commit(s) and 5 issue activity found in the last 90 days -- score normalized to 10
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Code-Review⚠️ 2Found 4/17 approved changesets -- score normalized to 2
Binary-Artifacts🟢 10no binaries found in the repo
Packaging⚠️ -1packaging workflow not detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
Pinned-Dependencies⚠️ 0dependency not pinned by hash detected -- score normalized to 0
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Security-Policy⚠️ 0security policy file not detected
License🟢 10license file detected
Fuzzing⚠️ 0project is not fuzzed
Signed-Releases⚠️ -1no releases found
Branch-Protection⚠️ 0branch protection not enabled on development/release branches
SAST⚠️ 0SAST tool is not run on all commits -- score normalized to 0
cargo/signal-hook-tokio 0.4.0 🟢 4
Details
CheckScoreReason
Maintained🟢 1021 commit(s) and 5 issue activity found in the last 90 days -- score normalized to 10
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Code-Review⚠️ 2Found 4/17 approved changesets -- score normalized to 2
Binary-Artifacts🟢 10no binaries found in the repo
Packaging⚠️ -1packaging workflow not detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
Pinned-Dependencies⚠️ 0dependency not pinned by hash detected -- score normalized to 0
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Security-Policy⚠️ 0security policy file not detected
License🟢 10license file detected
Fuzzing⚠️ 0project is not fuzzed
Signed-Releases⚠️ -1no releases found
Branch-Protection⚠️ 0branch protection not enabled on development/release branches
SAST⚠️ 0SAST tool is not run on all commits -- score normalized to 0
cargo/signal-hook >= 0.4.3, < 0.5.0 UnknownUnknown
cargo/signal-hook-tokio >= 0.4.0, < 0.5.0 UnknownUnknown

Scanned Files

  • rust/Cargo.lock
  • rust/crates/greenbone-scanner-framework/Cargo.toml

@nichtsfrei nichtsfrei marked this pull request as ready for review February 13, 2026 11:09
@nichtsfrei nichtsfrei requested a review from a team as a code owner February 13, 2026 11:09
Copy link
Member

@jjnicola jjnicola left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested and I couldn't see the log messages. However, the signal is received and testing with println!() the message is shown.

@nichtsfrei nichtsfrei force-pushed the signal-handling branch 3 times, most recently from f863b05 to eb9e95b Compare March 11, 2026 10:48
@nichtsfrei nichtsfrei requested a review from jjnicola March 11, 2026 10:53
@nichtsfrei nichtsfrei enabled auto-merge (rebase) March 11, 2026 10:53
@nichtsfrei nichtsfrei merged commit 8c0f005 into main Mar 11, 2026
33 of 35 checks passed
@nichtsfrei nichtsfrei deleted the signal-handling branch March 11, 2026 15:11
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.

2 participants