重定向返回http:多个response.WriteHeader调用

时间:2017-08-24 19:41:18

标签: http go httprouter

我正在使用github的Jon Calhoun's Go MVC framework

框架使用julienschmidt/httprouter作为唯一依赖项。

我有一个类似于示例中的主要方法:

  func main() {
        //register routes
        router := httprouter.New()

        //default
        router.GET("/", controllers.Login.Perform(controllers.Login.Index))

        //login
        router.GET("/login", controllers.Login.Perform(controllers.Login.Login))
        router.POST("/login", controllers.Login.Perform(controllers.Login.PostLogin))

        //dashboard
        router.GET("/dashboard", controllers.Dashboard.Perform(controllers.Dashboard.Index))


        //listen and handle requests
        log.Fatal(http.ListenAndServe(":"+helpers.ReadConfig("port_http"), router))
    }

我在登录网址上发帖,并调用以下方法:

func (self LoginController) PostLogin(w http.ResponseWriter, r *http.Request, ps httprouter.Params) error {
    //create our api url
    var url = helpers.ReadConfig("api") + "login"
    //fill model to post
    login := models.LoginModel{
        Password: r.FormValue("password"),
        Email:    r.FormValue("username"),
    }
    //render json from model
    bytes, err := json.Marshal(login)
    if err != nil {
        panic(err)
    }
    //post to the API helpers
    var resp = helpers.ApiPost(url, r, string(bytes))
    //check response if successful
    if resp.Code != constants.ApiResp_Success {
        //TODO: Handle API Errors
        login.Password = ""
        errors := make(map[int]string)
        errors[1] = "Please provide valid credntials."
        login.Common = models.CommonModel{
            ErrorList: errors,
        }
        return views.Login.Index.Render(w, login, helpers.AcceptsGzip(r))
    }


    log.Println("---Redirect--")
    http.Redirect(w, r, "/dashboard", 307)
    log.Println("-----")
    return views.Dashboard.Index.Render(w, login, helpers.AcceptsGzip(r))
}

基本上,如果登录不正确,我会返回相同的视图。如果登录正确,我想重定向到另一个控制器中的另一个方法。

但是当我调用http.Redirect(w, r, "/dashboard", 307)时,它会返回以下错误:

http: multiple response.WriteHeader calls  

我不确定为什么会发生这种情况,但我怀疑它与我的侦听器调用Perform函数有关,它会创建一个http.handler,如下所示。

func (c *Controller) Perform(a Action) httprouter.Handle {
    return httprouter.Handle(
        func(w http.ResponseWriter, r *http.Request, ps httprouter.Params) {
            //set response headers
            //TODO: set appropriate responce headers
            w.Header().Set("Access-Control-Allow-Origin", "*")
            w.Header().Set("Cache-Control", "public, max-age=0")
            w.Header().Set("Token", "NOT-A-VALID-TOKEN")
            w.WriteHeader(200)
            if err := a(w, r, ps); err != nil {
                http.Error(w, err.Error(), http.StatusInternalServerError)
            }

        })
}

有没有人知道如何使用这个MVC框架重定向?或者有一次性解决方案?

1 个答案:

答案 0 :(得分:4)

http.ResponseWriter WriteHeader方法只能针对每个HTTP响应调用一次,原因可能是:您只能拥有一个响应代码,并且只能发送一次标头。

您看到的错误意味着它在同一响应中第二次被调用。

您的中间件调用:

        w.WriteHeader(200)

然后你的处理程序也会调用:

http.Redirect(w, r, "/dashboard", 307)
log.Println("-----")
return views.Dashboard.Index.Render(w, login, helpers.AcceptsGzip(r))

在知道响应的命运之后,你的中间件永远不应该调用WriteHeader

此外,在不知道您的特定MVC框架的情况下,似乎可能在您发送307状态后,您还告诉MVC框架呈现响应,这也可能再次调用WriteHeader

相关问题