StackStorm/st2

Add 'shell_escape' Jinja filter

Open

#3.564 aberto em 11 de jul. de 2017

Ver no GitHub
 (2 comments) (4 reactions) (1 assignee)Python (740 forks)batch import
complexity:easyfeaturegood first issue

Métricas do repositório

Stars
 (5.794 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Hello Team,

I have tried to use local shell command to print the long string contains special characters. However it failed with below error as the output contains non-standard characters. It would be good if we can add shell_escape Jinja filter: https://docs.stackstorm.com/reference/jinja.html#applying-filters-with-jinja to handle cases like that when "dangerous" string is passed to shell command.

Failure error: bash: line 10: ok:: command not found bash: line 12: TASK: command not found bash: line 13: {: Permission denied bash: line 14: msg:: command not found

Guia do colaborador

Direção de pesquisa
Examine a implementação existente do filtro Jinja na base de código do StackStorm (provavelmente em st2common/filters.py ou similar). Consulte a documentação vinculada para o padrão do filtro shell escape. Verifique se o responsável fez algum progresso ou se há um branch relacionado. Considere implementar um filtro que escape metacaracteres do shell usando shlex.quote do Python ou similar. Crie uma nova função de filtro e registre a.
Pilha de tecnologia
python
Domain
backenddevops
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.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
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
Python
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70