aboutsummaryrefslogtreecommitdiffhomepage
path: root/ui/oauth2_callback.go
diff options
context:
space:
mode:
authorGravatar Frédéric Guillot <fred@miniflux.net>2018-09-23 21:02:26 -0700
committerGravatar Frédéric Guillot <fred@miniflux.net>2018-09-23 21:02:26 -0700
commit9d08139f4363d3503398002bc82cb3746e3438cf (patch)
treeb09c1efb14445624e4a2771bf8cbcf3a9828ecf9 /ui/oauth2_callback.go
parent844680e57328034c20a2d2b84bd315b55fee9e23 (diff)
Improve request package and add more unit tests
Diffstat (limited to 'ui/oauth2_callback.go')
-rw-r--r--ui/oauth2_callback.go6
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"))