Descriptionjuju/api.go: actually handle error (bug #1312136)
(refactored from a change John made)
If there is a problem reading the .jenv file, it can return an error,
but we weren't paying attention to it. Instead we would end up getting a
panic() a few lines down when we go to use a *Config that is actually
nil.
addition by Nate:
Refactored the code to extract the logic for getting the config into a separate function, which cleans up the error handling significantly.
https://code.launchpad.net/~natefinch/juju-core/jameinel-1.18-panic-parsing-jenv-1312136/+merge/217084
(do not edit description out of merge proposal)
Patch Set 1 #Patch Set 2 : juju/api.go: actually handle error (bug #1312136) #
MessagesTotal messages: 4
|