diff --git a/bridge/launchpad/config.go b/bridge/launchpad/config.go index 8469dbd3d8b9e25e1ec2e693927b2cd67a3d0e97..11a465bea5d9e9d9cb12e8338983ffbe7c0439db 100644 --- a/bridge/launchpad/config.go +++ b/bridge/launchpad/config.go @@ -25,6 +25,14 @@ func (*Launchpad) Configure(repo repository.RepoCommon) (core.Configuration, err return conf, nil } +func (*Launchpad) ValidateConfig(conf core.Configuration) error { + if _, ok := conf[keyProject]; !ok { + return fmt.Errorf("missing %s key", keyProject) + } + + return nil +} + func promptProjectName() (string, error) { for { fmt.Print("Launchpad project name: ") @@ -44,7 +52,3 @@ func promptProjectName() (string, error) { return line, nil } } - -func (*Launchpad) ValidateConfig(conf core.Configuration) error { - return nil -} diff --git a/bridge/launchpad/launchpad_api.go b/bridge/launchpad/launchpad_api.go index 09e02bc5ee0f755cea36f1b94893990ae7c5fe15..849ef925accbe6978ccfc046d716c640053c085a 100644 --- a/bridge/launchpad/launchpad_api.go +++ b/bridge/launchpad/launchpad_api.go @@ -129,11 +129,12 @@ func (lapi *launchpadAPI) SearchTasks(project string) ([]LPBug, error) { return nil, err } - defer resp.Body.Close() - var result launchpadAnswer - if err := json.NewDecoder(resp.Body).Decode(&result); err != nil { + err = json.NewDecoder(resp.Body).Decode(&result) + _ = resp.Body.Close() + + if err != nil { return nil, err }