Infrastructure/caddy
0
Fork 0
mirror of https://github.com/caddyserver/caddy.git synced 2024-12-23 22:27:38 -05:00
Code Issues Activity

Support env vars in import; make sure it's a file

Browse source
This commit is contained in:
Matthew Holt 2016-08-17 17:03:21 -06:00
parent c5524b0bab
commit 8b8afd72d7
No known key found for this signature in database
GPG key ID: 0D97CC73664F4D03
1 changed files with 12 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
caddyfile/parse.go
View file

@ -201,11 +201,14 @@ func (p *parser) directives() error {
// other words, call Next() to access the first token that was
// imported.
func (p *parser) doImport() error {
// syntax check
// syntax checks
if !p.NextArg() {
return p.ArgErr()
}
importPattern := p.Val()
importPattern := replaceEnvVars(p.Val())
if importPattern == "" {
return p.Err("Import requires a non-empty filepath")
}
if p.NextArg() {
return p.Err("Import takes only one argument (glob pattern or file)")
}
@ -284,6 +287,13 @@ func (p *parser) doSingleImport(importFile string) ([]Token, error) {
return nil, p.Errf("Could not import %s: %v", importFile, err)
}
defer file.Close()
if info, err := file.Stat(); err != nil {
return nil, p.Errf("Could not import %s: %v", importFile, err)
} else if info.IsDir() {
return nil, p.Errf("Could not import %s: is a directory", importFile)
}
importedTokens := allTokens(file)
// Tack the filename onto these tokens so errors show the imported file's name