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.

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

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

GarboMuffin
Copy link
Member

@GarboMuffin GarboMuffin commented Jan 13, 2025

#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

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

@GarboMuffin
Copy link
Member Author

i think its a good name but what do you propose

@hammouda101010
Copy link

hammouda101010 commented Jan 23, 2025

i think its a good name but what do you propose

--pc-full-access

@CST1229
Copy link

CST1229 commented Jan 26, 2025

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

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

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

@hammouda101010
Copy link

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.

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.

4 participants