diff options
author | Frédéric Guillot <fred@miniflux.net> | 2018-09-23 21:02:26 -0700 |
---|---|---|
committer | Frédéric Guillot <fred@miniflux.net> | 2018-09-23 21:02:26 -0700 |
commit | 9d08139f4363d3503398002bc82cb3746e3438cf (patch) | |
tree | b09c1efb14445624e4a2771bf8cbcf3a9828ecf9 /ui/oauth2_callback.go | |
parent | 844680e57328034c20a2d2b84bd315b55fee9e23 (diff) |
Improve request package and add more unit tests
Diffstat (limited to 'ui/oauth2_callback.go')
-rw-r--r-- | ui/oauth2_callback.go | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/ui/oauth2_callback.go b/ui/oauth2_callback.go index 9f51d0a..1902d6e 100644 --- a/ui/oauth2_callback.go +++ b/ui/oauth2_callback.go @@ -23,21 +23,21 @@ func (c *Controller) OAuth2Callback(w http.ResponseWriter, r *http.Request) { printer := locale.NewPrinter(request.UserLanguage(r)) sess := session.New(c.store, request.SessionID(r)) - provider := request.Param(r, "provider", "") + provider := request.RouteStringParam(r, "provider") if provider == "" { logger.Error("[OAuth2] Invalid or missing provider") response.Redirect(w, r, route.Path(c.router, "login")) return } - code := request.QueryParam(r, "code", "") + code := request.QueryStringParam(r, "code", "") if code == "" { logger.Error("[OAuth2] No code received on callback") response.Redirect(w, r, route.Path(c.router, "login")) return } - state := request.QueryParam(r, "state", "") + state := request.QueryStringParam(r, "state", "") if state == "" || state != request.OAuth2State(r) { logger.Error(`[OAuth2] Invalid state value: got "%s" instead of "%s"`, state, request.OAuth2State(r)) response.Redirect(w, r, route.Path(c.router, "login")) |