From 2b1da2b0b38dfc5d5841266037c0c8b249eca1dd Mon Sep 17 00:00:00 2001
From: "Farmer.Chillax" <48387781+FarmerChillax@users.noreply.github.com>
Date: Thu, 21 Mar 2024 21:08:41 +0800
Subject: [PATCH] fix(context): make context Value method adhere to Go
 standards (#3897)

---
 context.go      | 6 +++++-
 context_test.go | 2 +-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/context.go b/context.go
index 0c73a49..3a9608d 100644
--- a/context.go
+++ b/context.go
@@ -43,6 +43,10 @@ const BodyBytesKey = "_gin-gonic/gin/bodybyteskey"
 // ContextKey is the key that a Context returns itself for.
 const ContextKey = "_gin-gonic/gin/contextkey"
 
+type ContextKeyType int
+
+const ContextRequestKey ContextKeyType = 0
+
 // abortIndex represents a typical value used in abort functions.
 const abortIndex int8 = math.MaxInt8 >> 1
 
@@ -1225,7 +1229,7 @@ func (c *Context) Err() error {
 // if no value is associated with key. Successive calls to Value with
 // the same key returns the same result.
 func (c *Context) Value(key any) any {
-	if key == 0 {
+	if key == ContextRequestKey {
 		return c.Request
 	}
 	if key == ContextKey {
diff --git a/context_test.go b/context_test.go
index 089047c..9c1717e 100644
--- a/context_test.go
+++ b/context_test.go
@@ -1985,7 +1985,7 @@ func TestContextGolangContext(t *testing.T) {
 	ti, ok := c.Deadline()
 	assert.Equal(t, ti, time.Time{})
 	assert.False(t, ok)
-	assert.Equal(t, c.Value(0), c.Request)
+	assert.Equal(t, c.Value(ContextRequestKey), c.Request)
 	assert.Equal(t, c.Value(ContextKey), c)
 	assert.Nil(t, c.Value("foo"))