empty config.yaml causes panic
Bug #1199432 reported by
William Reade
This bug affects 2 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
juju-core |
Fix Released
|
High
|
Matthew Williams |
Bug Description
charm.ReadConfig unmarshals the content to (*Config)(nil), without error, and then tries to use .Options without checking. We ought to report an invalid config file instead of panicing.
Related branches
lp:~mattyw/juju-core/1199432
- Juju Engineering: Pending requested
-
Diff: 29 lines (+11/-0)2 files modifiedcharm/config.go (+3/-0)
charm/config_test.go (+8/-0)
Changed in juju-core: | |
status: | In Progress → Fix Committed |
Changed in juju-core: | |
milestone: | none → 1.14.0 |
status: | Fix Committed → Fix Released |
To post a comment you must log in.