Miserlou/Zappa

Check Config Files for Allowed Values

Open

#1 165 ouverte le 11 oct. 2017

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Python (1 286 forks)batch import
hacktoberfesthelp wanted

Métriques du dépôt

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

Description

Context

Was not seeing my remote environment variables because my config value was "remote_evn". Mistake on my part, but it would be nice if zappa checked config files to ensure that values being set were even values used by zappa.

Expected Behavior

When a user gives a config value not used by Zappa, it tells the user and suggests that it might be a typo. Just lowers the learning curve slightly and improves QOL.

Actual Behavior

Users are allowed to wallow in their mistakes

Possible Fix

Keep a white list of config keys and test against those.

Guide contributeur

Direction de recherche
Le problème suggère d'ajouter une liste blanche de clés de configuration valides dans la validation de configuration de Zappa. Examinez les fichiers comme 'zappa/cli.py' ou 'zappa/settings.py' où l'analyse de la configuration a lieu. Les mainteneurs n'ont pas fourni de conseils supplémentaires ; la correction proposée est de maintenir un ensemble de clés autorisées et de vérifier les clés fournies par l'utilisateur, en émettant un avertissement ou une erreur pour les clés inconnues.
Stack technique
python
Domaine
backend
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.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
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
PythonGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85