Skip to content

fix(execd): import mitm CA to NSS; set Node/npm trust for transparent egress#776

Merged
ninan-nn merged 1 commit intoalibaba:mainfrom
Pangjiping:hotfix/execd-ca
Apr 24, 2026
Merged

fix(execd): import mitm CA to NSS; set Node/npm trust for transparent egress#776
ninan-nn merged 1 commit intoalibaba:mainfrom
Pangjiping:hotfix/execd-ca

Conversation

@Pangjiping
Copy link
Copy Markdown
Collaborator

@Pangjiping Pangjiping commented Apr 24, 2026

Summary

When OPENSANDBOX_EGRESS_MITMPROXY_TRANSPARENT is set and the mitm CA PEM
is available:

  • Add trust_mitm_ca_nss: import the CA into $HOME/.pki/nssdb (Chrome uses NSS, not only the system store) via certutil; support sql/dbm and initialize an empty SQL DB if needed.
  • Export NODE_EXTRA_CA_CERTS to the same PEM for npm/Node TLS behind interception.
  • Install nss-tools in the execd image so certutil is available in Alpine.
  • Fix the bootstrap log to say 30s (the wait loop is 30 iterations).

Testing

  • Not run (explain why)
  • Unit tests
  • Integration tests
  • e2e / manual verification

Breaking Changes

  • None
  • Yes (describe impact and migration path)

Checklist

  • Linked Issue or clearly described motivation
  • Added/updated docs (if needed)
  • Added/updated tests (if needed)
  • Security impact considered
  • Backward compatibility considered

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d127fc4167

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread components/execd/bootstrap.sh Outdated
Copy link
Copy Markdown
Collaborator

@ninan-nn ninan-nn left a comment

Choose a reason for hiding this comment

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

LGTM

@ninan-nn ninan-nn merged commit d4044c1 into alibaba:main Apr 24, 2026
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working component/execd

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants