From 96b59b4c1f6d6ac09d657b9dcf55e4ad8c990c0c Mon Sep 17 00:00:00 2001
From: Chris Lu <chris.lu@gmail.com>
Date: Thu, 19 Feb 2026 17:28:58 -0800
Subject: [PATCH] comments

---
 weed/s3api/s3tables/iam.go | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/weed/s3api/s3tables/iam.go b/weed/s3api/s3tables/iam.go
index 655f49bae..08586b386 100644
--- a/weed/s3api/s3tables/iam.go
+++ b/weed/s3api/s3tables/iam.go
@@ -213,6 +213,12 @@ func buildIAMRequestContext(r *http.Request, claims map[string]interface{}) map[
 	return ctx
 }
 
+// getIdentityStructValue fetches the identity struct held in the request context.
+// The identity is expected to be a pointer to a struct with the fields used by
+// the reflection helpers (PrincipalArn string, PolicyNames []string,
+// Claims map[string]interface{}).
+// This helper centralizes the nil-check and ptr-deref logic so callers focus on
+// reading the specific fields they need.
 func getIdentityStructValue(r *http.Request) (reflect.Value, bool) {
 	identityRaw := s3_constants.GetIdentityFromContext(r)
 	if identityRaw == nil {
@@ -227,8 +233,3 @@ func getIdentityStructValue(r *http.Request) (reflect.Value, bool) {
 	}
 	return val, true
 }
-
-// The identity structure is expected to be a pointer to a struct with the
-// reflection fields used below (PrincipalArn string, PolicyNames []string,
-// Claims map[string]interface{}). This helper centralizes the nil-check / ptr-deref
-// logic so the callers can focus on reading the proper field.