Skip to content

fix(packaging): enable mTLS for local packages#1271

Merged
drew merged 5 commits intomainfrom
add-mtls
May 8, 2026
Merged

fix(packaging): enable mTLS for local packages#1271
drew merged 5 commits intomainfrom
add-mtls

Conversation

@drew
Copy link
Copy Markdown
Collaborator

@drew drew commented May 8, 2026

Summary

Enable TLS and client certificate auth for package-managed local gateways on Homebrew and Debian. Both packages now bootstrap local certs through openshell-gateway generate-certs --output-dir and register https://127.0.0.1:17670.

Related Issue

Addresses Linear OS-177

Changes

  • Configure the Debian user service with TLS server certs, sandbox guest TLS bundle paths, and an HTTPS callback endpoint.
  • Generate Homebrew local PKI during post_install with the shared gateway cert helper and update service environment and caveats.
  • Update install-dev and Debian package smoke registration to use the openshell HTTPS/mTLS local gateway.
  • Reuse existing local PKI when generate-certs --output-dir is re-run so CLI mTLS copies stay paired with the gateway certs.

Testing

  • cargo test -p openshell-server certgen
  • Generated Homebrew formula syntax check
  • Shell syntax checks for installer scripts
  • mise run pre-commit after moving the worktree: Python/docs/Helm/Rust check phases passed; Rust workspace tests then failed in openshell-ocsf because stale Cargo artifacts still referenced the old worktree path /Users/anewberry/dev/openshell.security-fixes.

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)

@drew drew requested review from a team, derekwaynecarr, maxamillion and mrunalp as code owners May 8, 2026 17:24
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 8, 2026

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented May 8, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

TaylorMutch
TaylorMutch previously approved these changes May 8, 2026
@drew drew merged commit daa2a36 into main May 8, 2026
23 checks passed
@drew drew deleted the add-mtls branch May 8, 2026 20:02
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