Gin: [WARNING] Headers were already written. Wanted to override status code 400 with 201

[dgsi]

It is working fine with localhost, but when i uploaded the app to aws ec2 i'm getting this warning. Records are created in database but i wasn't able to return the 201 status.

this is my code

//insert record to database
result := handler.db.Create(&newRoom)

//check if record is created
if result.RowsAffected > 0 {
c.JSON(http.StatusCreated, newRoom)
} else {
//display error message
errMsg := &ErrMsg{Message: result.Error.Error()}
c.JSON(http.StatusBadRequest,errMsg)
c.AbortWithStatus(http.StatusBadRequest)
}

[kongfei605]

try adding “return” statement at the place you want to terminate. Just like as this :
...
c.JSON(http.StatusCreated, newRoom)
return
...
c.JSON(http.StatusBadRequest,errMsg)
c.AbortWithStatus(http.StatusBadRequest)
return

[LeBronoMars]

not working :(

[tsirolnik]

You call -

c.JSON(http.StatusBadRequest,errMsg)
c.AbortWithStatus(http.StatusBadRequest)

c.JSON sends http.StatusBadRequest and then you send http.StatusBadRequest again with the call to AbortWithStatus.
You should only call one of them. Use return as the exit statement.

[LeBronoMars]

I have already figured this out.. this code c.Bind(&newIncident) throws an error but i'm not checking it.
now my code is err := c.Bind(&newIncident) and found out that i'm missing to set a value for a specific field in my POST request.

[stxml]

This was caused by PR #587. Fix is in #633.

You could use this until it's fixed.

func BindJSON(c *gin.Context, obj interface{}) error {
    if err := binding.JSON.Bind(c.Request, obj); err != nil {
        c.Error(err).SetType(gin.ErrorTypeBind)
        return err
    }
    return nil
}

[javierprovecho]

Check #855