Skip to content

[Security] Stage the dev-link sudoers draft out of world-writable /tmp - #10507

Open
Chessing234 wants to merge 1 commit into
omacom:quattrofrom
Chessing234:security/dev-link-sudoers-stage
Open

[Security] Stage the dev-link sudoers draft out of world-writable /tmp#10507
Chessing234 wants to merge 1 commit into
omacom:quattrofrom
Chessing234:security/dev-link-sudoers-stage

Conversation

@Chessing234

Copy link
Copy Markdown
Contributor

Summary

  • omarchy-dev-link wrote the sudoers drop-in with bare mktemp before visudo -cf and sudo install
  • That draft names the checkout path and is parsed as sudoers; it does not belong in world-writable /tmp
  • Stage it under $XDG_RUNTIME_DIR (or a 0700 /tmp/omarchy-$UID fallback)

Test plan

  • bash test/shell.d/dev-link-test.sh
  • omarchy dev link /path --no-reboot and confirm the staged file is not a /tmp/tmp.* path

Made with Cursor

@Chessing234

Copy link
Copy Markdown
Contributor Author

@dhh @ryanrhughes ready for review

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