usebruno/bruno

Consider using stronger ACL on Environment files

Open

#2.016 aberto em 4 de abr. de 2024

Ver no GitHub
 (1 comment) (1 reaction) (0 assignees)JavaScript (2.403 forks)batch import
good first issuehelp wantedmodule-environmentsmodule-filesystemmodule-security

Métricas do repositório

Stars
 (43.787 stars)
Métricas de merge de PR
 (Mesclagem média 6d 21h) (96 fundiu PRs em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Localize o código onde os arquivos de ambiente são criados (provavelmente no diretório environments). Altere as permissões de criação do arquivo do padrão (por exemplo, 644) para 600 usando fs.chmod ou métodos apropriados do sistema de arquivos Node.js. Certifique se de que o aplicativo ainda funcione corretamente com as novas permissões.
Pilha de tecnologia
javascript
Domain
security
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
None
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85