diff --git a/cleanenv.go b/cleanenv.go index 5dd5e7b..a64dcaf 100644 --- a/cleanenv.go +++ b/cleanenv.go @@ -333,7 +333,10 @@ func readStructMetadata(cfgRoot interface{}) ([]structMeta, error) { // process nested structure (except of supported ones) if fld := s.Field(idx); fld.Kind() == reflect.Struct { - + //skip unexported + if !fld.CanInterface() { + continue + } // add structure to parsing stack if _, found := validStructs[fld.Type()]; !found { prefix, _ := fType.Tag.Lookup(TagEnvPrefix) diff --git a/cleanenv_test.go b/cleanenv_test.go index b83cce0..d51490c 100644 --- a/cleanenv_test.go +++ b/cleanenv_test.go @@ -1187,3 +1187,26 @@ func TestTimeLocation(t *testing.T) { t.Errorf("wrong location pointers: got %p, want %p", S.Location, want) } } + +func TestSkipUnexportedField(t *testing.T) { + conf := struct { + Database struct { + Host string `yaml:"host" env:"DB_HOST" env-description:"Database host"` + Port string `yaml:"port" env:"DB_PORT" env-description:"Database port"` + } `yaml:"database"` + server struct { + Host string `yaml:"host" env:"SRV_HOST,HOST" env-description:"Server host" env-default:"localhost"` + Port string `yaml:"port" env:"SRV_PORT,PORT" env-description:"Server port" env-default:"8080"` + } `yaml:"server"` + }{} + + if err := ReadConfig("example/simple_config/config.yml", &conf); err != nil { + t.Fatal(err) + } + if conf.server.Host != "" || conf.server.Port != "" { + t.Fatal("unexpect value on unexported fields") + } + if conf.Database.Host == "" || conf.Database.Port == "" { + t.Fatal("expect value on exported fields") + } +}