Skip to content

Add --give-projects-full-access-to-my-computer-including-ability-to-install-malware - #1112

Draft
GarboMuffin wants to merge 1 commit into
masterfrom
node-integration
Draft

GarboMuffin wants to merge 1 commit into
masterfrom
node-integration

Conversation

@GarboMuffin

@GarboMuffin GarboMuffin commented Jan 13, 2025 •

Copy link
Copy Markdown
Member

#1110

Must be enabled by providing a command line flag and user must click agree to a prompt that looks like:

image

@hammouda101010

Copy link
Copy Markdown

can you choose another name other than --give-projects-full-access-to-my-computer-including-ability-to-install-malware

@GarboMuffin

Copy link
Copy Markdown
Member Author

i think its a good name but what do you propose

@hammouda101010

hammouda101010 commented Jan 23, 2025 •

Copy link
Copy Markdown

i think its a good name but what do you propose

--pc-full-access

@CST1229

CST1229 commented Jan 26, 2025 •

Copy link
Copy Markdown

i think its a good name but what do you propose

--pc-full-access

i think the intent of the long name is to let users ABSOLUTELY KNOW that they're enabling a dangerous option that they really shouldn't use

@hammouda101010

Copy link
Copy Markdown

i think its a good name but what do you propose

--pc-full-access

i think the intent of the long name is to let users ABSOLUTELY KNOW that they're enabling a dangerous option that they really shouldn't use

this name is much better and shorter anyways

@Brackets-Coder

Copy link
Copy Markdown
Member

I think you should keep it as the long name I agree with @CST1229

@hammouda101010

Copy link
Copy Markdown

I think you should keep it as the long name I agree with @CST1229

most command line flags are meant to be short and easy to understand. but this the most ridiculous name for a command line flag. if all of them where as long as this, it would be super frustrating, and might even break the PATH limit.

on the other hand, it alerts users to ABSOLUTLEY NOT use this flag for security.

@MatheusSilva11112009

Copy link
Copy Markdown

i'm not sure if that's a good idea

This branch has not been deployed

No deployments
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.

5 participants