Add better error messages for parsing response body
This commit is contained in:
parent
88934bbbcc
commit
dd14b103e3
|
|
@ -9,6 +9,7 @@ import (
|
||||||
"net"
|
"net"
|
||||||
"net/http"
|
"net/http"
|
||||||
"net/url"
|
"net/url"
|
||||||
|
"strings"
|
||||||
|
|
||||||
"linkheader"
|
"linkheader"
|
||||||
|
|
||||||
|
|
@ -151,17 +152,42 @@ func Authorize(me *url.URL, endpoints Endpoints, clientID, scope string) (TokenR
|
||||||
reqValues.Add("client_id", clientID)
|
reqValues.Add("client_id", clientID)
|
||||||
reqValues.Add("me", me.String())
|
reqValues.Add("me", me.String())
|
||||||
|
|
||||||
res, err := http.PostForm(endpoints.TokenEndpoint, reqValues)
|
req, err := http.NewRequest(http.MethodPost, endpoints.TokenEndpoint, strings.NewReader(reqValues.Encode()))
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return tokenResponse, err
|
return tokenResponse, err
|
||||||
}
|
}
|
||||||
|
|
||||||
|
res, err := http.DefaultClient.Do(req)
|
||||||
|
if err != nil {
|
||||||
|
return tokenResponse, err
|
||||||
|
}
|
||||||
defer res.Body.Close()
|
defer res.Body.Close()
|
||||||
|
|
||||||
|
if res.StatusCode != 200 {
|
||||||
|
return TokenResponse{}, fmt.Errorf("status code %d, instead of 200", res.StatusCode)
|
||||||
|
}
|
||||||
|
|
||||||
|
if strings.HasPrefix(res.Header.Get("content-type"), "application/json") {
|
||||||
dec := json.NewDecoder(res.Body)
|
dec := json.NewDecoder(res.Body)
|
||||||
err = dec.Decode(&tokenResponse)
|
err = dec.Decode(&tokenResponse)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return tokenResponse, err
|
return tokenResponse, fmt.Errorf("error while parsing response body with content-type %s as json: %s", res.Header.Get("content-type"), err)
|
||||||
|
}
|
||||||
|
} else if strings.HasPrefix(res.Header.Get("content-type"), "application/x-www-form-urlencoded") {
|
||||||
|
body, err := ioutil.ReadAll(res.Body)
|
||||||
|
if err != nil {
|
||||||
|
return tokenResponse, fmt.Errorf("error while reading response body with content-type %s: %s", res.Header.Get("content-type"), err)
|
||||||
|
}
|
||||||
|
|
||||||
|
values, err := url.ParseQuery(string(body))
|
||||||
|
if err != nil {
|
||||||
|
return tokenResponse, fmt.Errorf("error while parsing response body with content-type %s as application/x-www-form-urlencoded: %s", res.Header.Get("content-type"), err)
|
||||||
|
}
|
||||||
|
|
||||||
|
tokenResponse.Me = values.Get("me")
|
||||||
|
tokenResponse.AccessToken = values.Get("token")
|
||||||
|
tokenResponse.TokenType = values.Get("token_type")
|
||||||
|
tokenResponse.Scope = values.Get("scope")
|
||||||
}
|
}
|
||||||
|
|
||||||
return tokenResponse, nil
|
return tokenResponse, nil
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue
Block a user