DataRailroad

BindJSON vs ShouldBindJSON

BindJSON 은 확인해보면 *http.Request 를 사용하는 것을 확인할 수 있습니다. 그렇기 때문에 error 가 발생할 때 Request status code 가 400으로 변경되고 content-type 이 text/plain; charset=utf-8 로 변경됩니다. 이러한 점 때문에 ShouldBindJSON 사용을 권장하기도 합니다.

근데 테스트나 글 찾아보면 ShouldBindJSON 은 http.request 를 건들지 않는다고 하는데, Binding interface 까지 가보면 결국 BindJSON 과 같이 *Context.Request 를 호출해서 사용하는 걸 확인할 수 있습니다.

정확하게 문제에 대해서 이야기 한 사람 내용 입니다.

[이슈 설명] 현재, 함수 ctx.Bind, ctx.BindJSON, ctx.BindQuery는 모두 ctx.MustBindWith를 사용합니다. 제 의견으로는 현재의 ctx.MustBindWith의 구현에는 다음과 같은 단점이 있습니다. 이것은 이미 이전에 논의되었으며 여러 문제와 스레드를 크롤링하지 않기 위해 여기에 포함시킴을 양해 부탁드립니다.

1. 제어 부족
   • HTTP 상태 코드를 400으로 설정하며 개발자는 다른 HTTP 상태 코드(ex: 422, http.StatusUnprocessableEntity)를 설정할 수 없습니다.
   • 이것은 핸들러 체인을 중단합니다. 개발자는 결과 오류를 중단할 가치가 있는지 또는 조화될 수 있는지 결정할 수 있어야 합니다.
   • Bind JSON 및 Query에서 논의한대로 반복된 Bind를 사용할 수 없습니다. 개발자는 쿼리 매개변수와 JSON 본문을 별도로 바인딩하고 싶을 수 있습니다.
2. 가능한 잘못된 Content-type 헤더
   • Content-type을 text/plain으로 설정합니다.
   • ctx.JSON 또는 수동으로 콘텐츠 유형 헤더를 설정하려 해도 도움이 되지 않습니다. 대신 다음과 같은 경고가 표시됩니다: [GIN-debug] [WARNING] Headers were already written. Wanted to override status code 400 with 422

[제안] 역호환성을 유지하기 위해 기존 바인드 방법의 동작을 변경할 수 없습니다. 따라서 이 PR은 ShouldBind 대응 항목을 추가합니다 - ctx.ShouldBind, ctx.ShouldBindJSON, ctx.ShouldBindQuery.

Readme 파일은 아직 업데이트하지 않았으며 여기서의 토론 결과에 따라 업데이트할 것을 제안합니다. readme의 모든 예제를 Should 동등 항목을 사용하도록 전환하는 것을 제안합니다. readme에 있는 모든 예제는 ShouldBind 방법을 사용하도록 업데이트되었습니다. 또한 Bind와 ShouldBind 방법의 차이를 설명하는 섹션을 추가했습니다.

참조

• Use ShouldBindJSON instead of BindJSON
• https://github.com/iden3/go-iden3-servers/issues/6
• 400 (Bad Request) gets automatically send when c.BindJSON(&struct) gets error #782 https://github.com/gin-gonic/gin/issues/782
• ShouldBind counterparts for Bind methods https://github.com/gin-gonic/gin/pull/1047