Skip to content

NixOS first integration#8

Merged
Ravenink merged 16 commits intomainfrom
NixOS
Feb 16, 2026
Merged

NixOS first integration#8
Ravenink merged 16 commits intomainfrom
NixOS

Conversation

@Ravenink
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings February 16, 2026 20:27
@Ravenink Ravenink closed this Feb 16, 2026
@github-actions
Copy link

github-actions bot commented Feb 16, 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.

License Issues

.github/workflows/nix-integration.yml

PackageVersionLicenseIssue Type
DeterminateSystems/nix-installer-action19.*.*NullUnknown License
actions/checkout6.*.*NullUnknown License

OpenSSF Scorecard

PackageVersionScoreDetails
actions/DeterminateSystems/nix-installer-action 19.*.* 🟢 5.6
Details
CheckScoreReason
Code-Review🟢 10all changesets reviewed
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Binary-Artifacts🟢 10no binaries found in the repo
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
Packaging⚠️ -1packaging workflow not detected
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Maintained🟢 911 commit(s) and 0 issue activity found in the last 90 days -- score normalized to 9
License🟢 10license file detected
Signed-Releases⚠️ -1no releases found
Fuzzing⚠️ 0project is not fuzzed
Branch-Protection🟢 5branch protection is not maximal on development and all release branches
Pinned-Dependencies⚠️ 0dependency not pinned by hash detected -- score normalized to 0
Security-Policy⚠️ 0security policy file not detected
Vulnerabilities🟢 91 existing vulnerabilities detected
SAST⚠️ 0SAST tool is not run on all commits -- score normalized to 0
actions/actions/checkout 6.*.* 🟢 6.5
Details
CheckScoreReason
Code-Review🟢 10all changesets reviewed
Maintained🟢 57 commit(s) and 0 issue activity found in the last 90 days -- score normalized to 5
Binary-Artifacts🟢 10no binaries found in the repo
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Packaging⚠️ -1packaging workflow not detected
Fuzzing⚠️ 0project is not fuzzed
License🟢 10license file detected
Pinned-Dependencies🟢 3dependency not pinned by hash detected -- score normalized to 3
Signed-Releases⚠️ -1no releases found
Security-Policy🟢 9security policy file detected
Branch-Protection🟢 6branch protection is not maximal on development and all release branches
Vulnerabilities🟢 82 existing vulnerabilities detected
SAST🟢 8SAST tool detected but not run on all commits

Scanned Files

  • .github/workflows/nix-integration.yml

@Ravenink Ravenink reopened this Feb 16, 2026
@Ravenink Ravenink merged commit 17299cc into main Feb 16, 2026
18 of 19 checks passed
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR integrates NixOS deployment support alongside Docker Compose for the DecentraLabs Gateway. It introduces a Nix flake with a NixOS module for declarative service management, renames environment variables for clarity (consolidating access tokens), adds Lite mode for JWT delegation, and updates documentation to reflect these changes.

Changes:

  • Added NixOS module and flake configuration for declarative deployment
  • Renamed SECURITY_ACCESS_TOKEN to TREASURY_TOKEN and OPS_SECRET to LAB_MANAGER_TOKEN throughout the codebase
  • Introduced Lite mode that syncs JWT public keys from external issuers and disables local auth/treasury endpoints

Reviewed changes

Copilot reviewed 49 out of 50 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
flake.nix NixOS flake outputs with module and host configuration
nix/nixos-module.nix NixOS service module for compose-managed deployment
nix/lab-gateway-docker.nix Docker Compose wrapper script for Nix integration
nix/hosts/gateway.nix Default host configuration for gateway deployments
openresty/init-ssl.sh JWT key sync logic for Lite mode from external issuer
openresty/lua/init.lua Lite mode detection and configuration
openresty/lua/treasury_access.lua Renamed from internal_access.lua with Lite mode blocks
openresty/lua/admin_access.lua Updated to use TREASURY_TOKEN only
openresty/lab_access.conf Lite mode endpoint restrictions and mode detection endpoint
setup.sh / setup.bat Updated token names and added Lite mode ISSUER prompt
docker-compose.yml Environment variable renames for treasury and lab manager tokens
Various test files Token name updates and new treasury/admin access specs
Documentation files Installation guides, tutorials, and audit notes
Comments suppressed due to low confidence (1)

setup.sh:1

  • The placeholder detection logic appears in both setup.sh and setup.bat. Consider documenting or consolidating the list of disallowed placeholder values to ensure consistency across both scripts.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

auto_refresh_jwt_public_key &

exec /usr/local/openresty/bin/openresty -g "daemon off;"

Copy link

Copilot AI Feb 16, 2026

Choose a reason for hiding this comment

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

The script ends with a blank line after the exec command. While not harmful, the blank line after exec is unreachable code since exec replaces the current process. Consider removing line 403 for clarity.

Suggested change

Copilot uses AI. Check for mistakes.
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