diff options
author | Joram Wilander <jwawilander@gmail.com> | 2016-09-06 08:29:14 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2016-09-06 08:29:14 -0400 |
commit | dda0ea355283623cf9bf3f5d16f3a3c3033106ae (patch) | |
tree | 83e51c84873f38a05407592b050c726ff76fb6b3 /api | |
parent | a3ea88441986c4c655e4590b96ebdc2712ae7028 (diff) | |
download | chat-dda0ea355283623cf9bf3f5d16f3a3c3033106ae.tar.gz chat-dda0ea355283623cf9bf3f5d16f3a3c3033106ae.tar.bz2 chat-dda0ea355283623cf9bf3f5d16f3a3c3033106ae.zip |
PLT-3875 Update error page to use Markdown and add error for missing OAuth code (#3934)
* Update error page to use Markdown and add error for missing OAuth code
* Add line breaks to error messages
This is to make the error message easier to read.
* Update en.json
Diffstat (limited to 'api')
-rw-r--r-- | api/oauth.go | 15 |
1 files changed, 6 insertions, 9 deletions
diff --git a/api/oauth.go b/api/oauth.go index ef6c0a80b..b1c7675ff 100644 --- a/api/oauth.go +++ b/api/oauth.go @@ -4,7 +4,6 @@ package api import ( - "bytes" "crypto/tls" b64 "encoding/base64" "fmt" @@ -290,9 +289,12 @@ func completeOAuth(c *Context, w http.ResponseWriter, r *http.Request) { params := mux.Vars(r) service := params["service"] - l4g.Debug(utils.T("api.oauth.receive_redirect.debug", map[string]interface{}{"URL": r.URL.String()})) - code := r.URL.Query().Get("code") + if len(code) == 0 { + c.Err = model.NewLocAppError("completeOAuth", "api.oauth.complete_oauth.missing_code.app_error", map[string]interface{}{"service": strings.Title(service)}, "URL: "+r.URL.String()) + return + } + state := r.URL.Query().Get("state") uri := c.GetSiteURL() + "/signup/" + service + "/complete" @@ -779,12 +781,7 @@ func AuthorizeOAuthUser(service, code, state, redirectUri string) (io.ReadCloser if resp, err := client.Do(req); err != nil { return nil, "", nil, model.NewLocAppError("AuthorizeOAuthUser", "api.user.authorize_oauth_user.token_failed.app_error", nil, err.Error()) } else { - // temporarily read the raw body for debugging purposes - respBody, _ = ioutil.ReadAll(resp.Body) - - reader := bytes.NewReader(respBody) - - ar = model.AccessResponseFromJson(reader) + ar = model.AccessResponseFromJson(resp.Body) defer func() { ioutil.ReadAll(resp.Body) resp.Body.Close() |