Skip to content

Run Linux drop-sudo with reduced process credentials - #122

Open
winston-openai wants to merge 4 commits into
openai:mainfrom
winston-openai:dev/winston/clean-drop-sudo-remake
Open

Run Linux drop-sudo with reduced process credentials#122
winston-openai wants to merge 4 commits into
openai:mainfrom
winston-openai:dev/winston/clean-drop-sudo-remake

Conversation

@winston-openai

Copy link
Copy Markdown

Summary

  • launch Codex under the existing Linux runner UID and primary GID while clearing supplementary groups and process capabilities
  • set no_new_privs and hold the prompt until sudo removal has completed and been verified
  • preserve the runner's home, checkout ownership, Git configuration, caches, and output-file behavior
  • keep promptless setup available while warning that later direct Codex commands do not pass through the action's reduced process identity
  • add focused Linux coverage for both successful credential reduction and fail-closed startup

Compatibility

  • macOS behavior remains unchanged
  • Linux drop-sudo requires a non-root runner with passwordless sudo and /usr/bin/setpriv; GitHub-hosted Ubuntu runners provide this setup
  • no system accounts, filesystem ACLs, Git safe-directory entries, or alternate home directories are introduced

Validation

  • pnpm run check
  • pnpm test
  • both privileged Linux cases passed in a disposable Node 22 environment
  • git diff --check
  • public-content scan for internal references and disclosure-specific terminology

@github-actions

github-actions Bot commented Jul 17, 2026

Copy link
Copy Markdown

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@winston-openai
winston-openai marked this pull request as ready for review July 17, 2026 06:28
@winston-openai

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

github-actions Bot added a commit that referenced this pull request Jul 17, 2026
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