Miserlou/Zappa

Check Config Files for Allowed Values

Open

#1165 aperta il 11 ott 2017

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)Python (1286 fork)batch import
hacktoberfesthelp wanted

Metriche repository

Star
 (11.903 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Il problema suggerisce di aggiungere una whitelist di chiavi di configurazione valide nella validazione della configurazione di Zappa. Esamina file come 'zappa/cli.py' o 'zappa/settings.py' dove avviene il parsing della configurazione. I manutentori non hanno fornito ulteriori indicazioni; la soluzione proposta è mantenere un insieme di chiavi consentite e verificare le chiavi fornite dall'utente, emettendo un avviso o un errore per chiavi sconosciute.
Tech stack
python
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85