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

Unix: Directories in ~/.dapr given 777 permisions. #1317

Closed
JoshVanL opened this issue Jun 14, 2023 · 2 comments · Fixed by #1375
Closed

Unix: Directories in ~/.dapr given 777 permisions. #1317

JoshVanL opened this issue Jun 14, 2023 · 2 comments · Fixed by #1375
Labels
area/cli kind/bug Something isn't working P1 size/XS 2 days work triaged/resolved The issue has been triaged
Milestone
v1.13

Comments

@JoshVanL
Copy link
Contributor

Expected Behavior

Directories in ~/.dapr are given 755 permissions.

drwxr-xr-x  3 josh users   5 Jun 14 13:43 bin
drwxr-xr-x  2 josh users   4 Jun 14 13:42 components

Actual Behavior

Directories in ~/.dapr are given 777 permissions.

drwxrwxrwx  3 josh users   5 Jun 14 13:43 bin
drwxrwxrwx  2 josh users   4 Jun 14 13:42 components

Steps to Reproduce the Problem

Release Note

RELEASE NOTE:
@JoshVanL JoshVanL added the kind/bug Something isn't working label Jun 14, 2023
@mukundansundar mukundansundar added area/cli P1 size/XS 2 days work triaged/resolved The issue has been triaged labels Jun 15, 2023
@mukundansundar mukundansundar added this to the v1.12 milestone Jun 15, 2023
@mukundansundar mukundansundar modified the milestones: v1.12, v1.13 Sep 15, 2023
@twinguy
Copy link
Contributor

twinguy commented Jan 19, 2024

Hi, first time contributor here, Quick question: Are there any obvious reasons the bin and components directories would be set to 777 or is this just a best practice oversight? If so, I will be glad to help with this issue.

@yaron2
Copy link
Member

yaron2 commented Jan 19, 2024

Oversight. cc @mukundansundar

@twinguy twinguy mentioned this issue Jan 19, 2024
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/cli kind/bug Something isn't working P1 size/XS 2 days work triaged/resolved The issue has been triaged
Projects
No open projects
Grace Hopper Conference 2023
Status: Done
Milestone
v1.13
Development

Successfully merging a pull request may close this issue.

Fix default installation directory permissions twinguy/cli
4 participants
@JoshVanL @yaron2 @twinguy @mukundansundar