Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

permission: drop process.permission.deny #47335

Merged
merged 1 commit into from
Apr 4, 2023
Merged

permission: drop process.permission.deny #47335

merged 1 commit into from
Apr 4, 2023

Conversation

RafaelGSS
Copy link
Member

As discussed previously in #47105 and #47269. We're dropping the process.permission.deny API to reduce the surface for a potential attack vector.

This feature isn't out of our plans, however, we reached a consensus to remove it for further discussion. Also note that this PR isn't a semver-major since the feature (permission model) wasn't released yet.

@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run. labels Mar 31, 2023
@RafaelGSS RafaelGSS force-pushed the drop-permission-deny branch from 4aff6eb to 19aacdd Compare March 31, 2023 10:53
@RafaelGSS RafaelGSS added the request-ci Add this label to start a Jenkins CI on a PR. label Mar 31, 2023
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Mar 31, 2023
@nodejs-github-bot
Copy link
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/50747/

@RafaelGSS RafaelGSS force-pushed the drop-permission-deny branch from 19aacdd to f779526 Compare March 31, 2023 15:21
@github-actions github-actions bot mentioned this pull request Apr 1, 2023
Open
38 tasks
@RafaelGSS RafaelGSS added the request-ci Add this label to start a Jenkins CI on a PR. label Apr 1, 2023
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Apr 1, 2023
@nodejs-github-bot
Copy link
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/50784/

@aduh95
Copy link
Contributor

aduh95 commented Apr 1, 2023

Seemingly related CI failures on Windows.

@github-actions github-actions bot mentioned this pull request Apr 2, 2023
Open
30 tasks
@RafaelGSS
Copy link
Member Author

Seemingly related CI failures on Windows.

Yes, I'll fix it today

@RafaelGSS RafaelGSS force-pushed the drop-permission-deny branch from f779526 to ea8d2dd Compare April 2, 2023 19:11
@RafaelGSS RafaelGSS added the request-ci Add this label to start a Jenkins CI on a PR. label Apr 2, 2023
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Apr 2, 2023
@nodejs-github-bot
Copy link
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/50825/

@RafaelGSS RafaelGSS force-pushed the drop-permission-deny branch from ea8d2dd to 2096325 Compare April 2, 2023 21:28
@RafaelGSS RafaelGSS added the request-ci Add this label to start a Jenkins CI on a PR. label Apr 2, 2023
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Apr 2, 2023
@nodejs-github-bot
Copy link
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/50835/

@github-actions github-actions bot mentioned this pull request Apr 3, 2023
Open
28 tasks
@RafaelGSS RafaelGSS force-pushed the drop-permission-deny branch from 2096325 to 84ad3bc Compare April 3, 2023 13:08
@RafaelGSS RafaelGSS added the request-ci Add this label to start a Jenkins CI on a PR. label Apr 3, 2023
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Apr 3, 2023
@nodejs-github-bot
Copy link
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/50859/

This was referenced Apr 3, 2023
Closed
Merged
@nodejs-github-bot
Copy link
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/50864/

@nodejs-github-bot
Copy link
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/50876/

@github-actions github-actions bot mentioned this pull request Apr 4, 2023
Open
29 tasks
@nodejs-github-bot
Copy link
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/50919/

@RafaelGSS RafaelGSS added the commit-queue Add this label to land a pull request using GitHub Actions. label Apr 4, 2023
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Apr 4, 2023
@nodejs-github-bot nodejs-github-bot merged commit 6fd147c into nodejs:main Apr 4, 2023
@nodejs-github-bot
Copy link
Collaborator

Landed in 6fd147c

@RafaelGSS RafaelGSS added backport-blocked-v16.x backport-blocked-v18.x PRs that should land on the v18.x-staging branch but are blocked by another PR's pending backport. labels Apr 4, 2023
@github-actions github-actions bot mentioned this pull request Apr 5, 2023
Open
26 tasks
@RafaelGSS RafaelGSS mentioned this pull request Apr 11, 2023
Merged
RafaelGSS added a commit that referenced this pull request Apr 13, 2023
@RafaelGSS
permission: drop process.permission.deny
b441b5d 
PR-URL: #47335
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Beth Griggs <bethanyngriggs@gmail.com>
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
@abscondment
Copy link

@RafaelGSS is there an issue tracking getting this change backported?

@RafaelGSS
Copy link
Member Author

@RafaelGSS is there an issue tracking getting this change backported?

Permission Model is only available on Node.js 20. There's no backport.

@RafaelGSS RafaelGSS added dont-land-on-v16.x dont-land-on-v18.x PRs that should not land on the v18.x-staging branch and should not be released in v18.x. and removed backport-blocked-v16.x backport-blocked-v18.x PRs that should land on the v18.x-staging branch but are blocked by another PR's pending backport. labels Jun 16, 2023
@tniessen tniessen added the permission Issues and PRs related to the Permission Model label Aug 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@marco-ippolito marco-ippolito marco-ippolito approved these changes

@aduh95 aduh95 aduh95 approved these changes

@BethGriggs BethGriggs BethGriggs approved these changes

@jasnell jasnell jasnell approved these changes

Assignees
No one assigned
Labels
c++ Issues and PRs that require attention from people who are familiar with C++. dont-land-on-v18.x PRs that should not land on the v18.x-staging branch and should not be released in v18.x. lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run. permission Issues and PRs related to the Permission Model
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@RafaelGSS @nodejs-github-bot @aduh95 @abscondment @jasnell @BethGriggs @marco-ippolito @tniessen