dotnet/msbuild

Provide a method to execute an external command without using a shell

Open

#399 ouverte le 10 déc. 2015

Voir sur GitHub
 (6 commentaires) (3 réactions) (0 assignés)C# (1 364 forks)batch import
help wantedneeds-designtriaged

Métriques du dépôt

Stars
 (5 062 stars)
Métriques de merge PR
 (Merge moyen 11j 7h) (125 PRs mergées en 30 j)

Description

The Exec task always spawns a shell to invoke the external command. This is convenient, but non-portable, insecure (though not exploitable here), and requires non-portable escaping.

A better solution is to provide a way to execute an external command without using a shell. It should have the following properties:

  • Lists are respected.
  • Any character can be passed to an external command. No manual escaping required.
  • Special characters in property values do not cause problems.
  • No shell is invoked, unless the external command is the shell.
  • It should be possible to pass multiple arguments to the shell
  • The syntax should be something other than requiring a seperate XML element per argument.

Guide contributeur

Direction de recherche
Examinez l'implémentation actuelle de la tâche Exec dans le code source de MSBuild, comprenez comment MSBuild exécute les tâches avec invocation du shell et concevez une nouvelle méthode (par exemple, une nouvelle tâche ou une syntaxe étendue) qui permet d'exécuter des commandes sans shell, prenant en charge les listes et évitant les problèmes d'échappement. Envisagez d'utiliser un tableau d'arguments plutôt qu'une chaîne de commande.
Stack technique
csharp
Domaine
build system
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
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.
Plutôt claire
Prérequis
C#
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40