usebruno/bruno

Consider using stronger ACL on Environment files

Open

#2016 aperta il 4 apr 2024

Vedi su GitHub
 (1 commento) (1 reazione) (0 assegnatari)JavaScript (2403 fork)batch import
good first issuehelp wantedmodule-environmentsmodule-filesystemmodule-security

Metriche repository

Star
 (43.787 star)
Metriche merge PR
 (Merge medio 6g 21h) (96 PR mergiate in 30 g)

Descrizione

Issue

When an Environment file is created, it is typically stored in the environments directory. On 'nix/BSD environments, those files are stored with world-readable perms (644 to be exact). While there is already some protection for sensitive data by using the "Secrets" checkbox, I could see people who accidentally/mistakenly still store sensitive creds and keys which could expose them.

I'd recommend you set an ACL for the Environment files to 600 by default. I can confirm that Bruno will continue to read and write to them just fine with those permissions set.

Guida contributor