keystonejs/keystone-classic

Make less files not publicly available

Aberta

#1.639 aberto em 2 de set. de 2015

Ver no GitHubGuia do colaborador
 (1 comentário) (0 reação) (0 responsável)JavaScript (2.288 forks)batch import
bughelp wanted

Métricas do repositório

Stars
 (14.656 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Every file under /public/styles/ is available through http when keystone is running.

Since *.less files could be considered as a part of the source code, could we move them to another location, e.g. /templates/styles/ to protect them from being available publicly?

It may be a good idea to keep /public/ folder only for files that are required during runtime, and keep source files elsewhere.

Guia do colaborador

Direção de pesquisa
Investigue a estrutura atual de arquivos em /public/styles/ e /templates/ para entender como o Keystone serve ativos estáticos. Identifique todas as referências a arquivos de estilo no aplicativo (por exemplo, em modelos, arquivos de layout ou configuração do servidor) que precisariam ser atualizadas após a mudança. Verifique se existem mecanismos atuais para impedir o acesso público a determinados tipos de arquivo e considere se uma solução mais geral (como .htaccess ou middleware) é preferível. Revise quaisquer comentários na issue para obter orientação dos mantenedores sobre a abordagem desejada.
Pilha de tecnologia
javascript
Domain
security
Tipo Issue
Manutenção
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
Keystone.js
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60