deno
fix(permissions): don't require --allow-run for process.kill on self
#34382
Merged
Merged
fix(permissions): don't require --allow-run for process.kill on self #34382
bartlomieju
merged
3 commits into
main
from
fix/process-kill-self-permission
fix(permissions): allow process.kill / Deno.kill on self without --al…
3c1e37e9
fibibot
commented
on 2026-05-25
fibibot
dismissed these changes
on 2026-05-25
test(process): update killPermissions for self-pid carveout
d51b70e0
Merge remote-tracking branch 'origin/main' into fix/process-kill-self…
a037104d
bartlomieju
dismissed their stale review
133 days ago
Addressed in commit d51b70e: killPermissions now asserts against a non-self pid (Deno.pid + 1) so the run-permission contract for other pids is preserved, while the self-pid carveout is exercised by the new killSelfNoPermissions unit test and the process_kill_self spec. CI is green on the current head.
bartlomieju
commented
on 2026-06-16
Login to write a write a comment.