StackStorm/st2

Add 'shell_escape' Jinja filter

Open

#3 564 ouverte le 11 juil. 2017

Voir sur GitHub
 (2 commentaires) (4 réactions) (1 assigné)Python (740 forks)batch import
complexity:easyfeaturegood first issue

Métriques du dépôt

Stars
 (5 794 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

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

Guide contributeur

Direction de recherche
Examinez l'implémentation existante du filtre Jinja dans la base de code de StackStorm (probablement dans st2common/filters.py ou similaire). Consultez la documentation liée pour le modèle de filtre shell escape. Vérifiez si l'assigné a fait des progrès ou s'il existe une branche associée. Envisagez d'implémenter un filtre qui échappe les métacaractères du shell en utilisant shlex.quote de Python ou similaire. Créez une nouvelle fonction de filtre et enregistrez la.
Stack technique
python
Domaine
backenddevops
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Python
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70