From 7f070584076c2d7e00a47dfdb3bac531b28524f0 Mon Sep 17 00:00:00 2001 From: mehrdadbn9 Date: Fri, 6 Feb 2026 15:42:49 +0330 Subject: [PATCH] Use http.StatusContinue constant instead of magic number 100 Replace magic number `100` with `http.StatusContinue` constant for better code clarity and maintainability in `bodyAllowedForStatus` function. Fixes #4489 --- context.go | 1 + context_test.go | 1 + 2 files changed, 2 insertions(+) diff --git a/context.go b/context.go index a00d1e55..4c71a15b 100644 --- a/context.go +++ b/context.go @@ -1056,6 +1056,7 @@ func (c *Context) requestHeader(key string) string { /************************************/ // bodyAllowedForStatus is a copy of http.bodyAllowedForStatus non-exported function. +// Use http.StatusContinue constant for better code clarity func bodyAllowedForStatus(status int) bool { switch { case status >= http.StatusContinue && status < http.StatusOK: diff --git a/context_test.go b/context_test.go index 41694585..408b45bc 100644 --- a/context_test.go +++ b/context_test.go @@ -1033,6 +1033,7 @@ func TestContextGetCookie(t *testing.T) { func TestContextBodyAllowedForStatus(t *testing.T) { assert.False(t, bodyAllowedForStatus(http.StatusProcessing)) assert.False(t, bodyAllowedForStatus(http.StatusNoContent)) + assert.False(t, bodyAllowedForStatus(http.StatusContinue)) assert.False(t, bodyAllowedForStatus(http.StatusNotModified)) assert.True(t, bodyAllowedForStatus(http.StatusInternalServerError)) }