sthope/gitea_templates
Template
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Use valid instead of allow in configWithTest (#101)

Browse Source 
The current validation using `allow` will allow any kind of string, not only the specified ones. 
`NODE_ENV=foo` is totally valid. Using `valid` instead should fix this.
This commit is contained in:
René Herrmann 2017-10-20 00:19:46 +02:00 committed by Vahid Panjganj
parent 07463964fc
commit 3ecb3d21db
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
configWithTest.sample.js
View File

@ -2,12 +2,12 @@ const joi = require('joi')
const envVarsSchema = joi.object({ const envVarsSchema = joi.object({
NODE_ENV: joi.string() NODE_ENV: joi.string()
.allow(['development', 'production', 'test', 'provision']) .valid(['development', 'production', 'test', 'provision'])
.required(), .required(),
PORT: joi.number() PORT: joi.number()
.required(), .required(),
LOGGER_LEVEL: joi.string() LOGGER_LEVEL: joi.string()
.allow(['error', 'warn', 'info', 'verbose', 'debug', 'silly']) .valid(['error', 'warn', 'info', 'verbose', 'debug', 'silly'])
.default('info'), .default('info'),
LOGGER_ENABLED: joi.boolean() LOGGER_ENABLED: joi.boolean()
.truthy('TRUE') .truthy('TRUE')