Invoke envOverrides before checkFilePerms (#1441)

* Invoke envOverrides before checkFilePerms

* Reverts changes to cmd/proxy/Dockerfile
This commit is contained in:
David Farrell
2019-11-07 11:27:29 -08:00
committed by Marwan Sulaiman
parent 402264a8f3
commit db84b52159
+5 -5
View File
@@ -190,6 +190,11 @@ func ParseConfigFile(configFile string) (*Config, error) {
return nil, err
}
// override values with environment variables if specified
if err := envOverride(&config); err != nil {
return nil, err
}
// Check file perms from config
if config.GoEnv == "production" {
if err := checkFilePerms(configFile, config.FilterFile); err != nil {
@@ -197,11 +202,6 @@ func ParseConfigFile(configFile string) (*Config, error) {
}
}
// override values with environment variables if specified
if err := envOverride(&config); err != nil {
return nil, err
}
// validate all required fields have been populated
if err := validateConfig(config); err != nil {
return nil, err