Conversation
…n have been combined into just one. Updated docs
Dependency ReviewThe following issues were found:
License Issues.github/workflows/nix-integration.yml
OpenSSF Scorecard
Scanned Files
|
There was a problem hiding this comment.
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_TOKENtoTREASURY_TOKENandOPS_SECRETtoLAB_MANAGER_TOKENthroughout 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;" | ||
|
|
There was a problem hiding this comment.
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.
No description provided.