Fix defer location and err check

This commit is contained in:
Joachim Hill-Grannec 2016-06-12 22:18:31 -07:00
parent 6f4c4a37dc
commit be1b75fd45

View file

@ -119,11 +119,13 @@ func (c *client) Login(res http.ResponseWriter, req *http.Request) (*model.User,
if err != nil { if err != nil {
return nil, err return nil, err
} }
defer response1.Body.Close()
contents, err := ioutil.ReadAll(response1.Body) contents, err := ioutil.ReadAll(response1.Body)
if err !=nil { if err !=nil {
return nil, err return nil, err
} }
defer response1.Body.Close()
var user User var user User
err = json.Unmarshal(contents, &user) err = json.Unmarshal(contents, &user)
if err != nil { if err != nil {
@ -264,6 +266,9 @@ func (*client) Status(*model.User, *model.Repo, *model.Build, string) error {
func (c *client) Netrc(user *model.User, r *model.Repo) (*model.Netrc, error) { func (c *client) Netrc(user *model.User, r *model.Repo) (*model.Netrc, error) {
u, err := url.Parse(c.URL) u, err := url.Parse(c.URL)
if err != nil {
return nil, err
}
//remove the port //remove the port
tmp := strings.Split(u.Host, ":") tmp := strings.Split(u.Host, ":")
var host = tmp[0] var host = tmp[0]