[Experiment] try to bypass issues with go 1.21.x

Co-authored-by: Jörn Dreyer <jfd@owncloud.com>

Signed-off-by: Christian Richter <crichter@owncloud.com>
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>

changelog/unreleased/go-1.21-compatability.md

@@ -0,0 +1,5 @@
+Bugfix: make appctx package compatible with go v1.21
+
+Backported fix from edge. See https://tip.golang.org/doc/go1.21#reflect
+
+https://github.com/cs3org/reva/pull/4214

pkg/appctx/ctxmap.go

@@ -41,20 +41,24 @@ func GetKeyValuesFromCtx(ctx context.Context) map[interface{}]interface{} {
 }
 
 func getKeyValue(ctx interface{}, m map[interface{}]interface{}) {
-	ctxVals := reflect.ValueOf(ctx).Elem()
-	ctxType := reflect.TypeOf(ctx).Elem()
+	// This is a dirty hack to run with go 1.21.x
+	reflectCtxValues := reflect.ValueOf(ctx)
+	if reflectCtxValues.Kind() != reflect.Struct {
+		ctxVals := reflectCtxValues.Elem()
+		ctxType := reflect.TypeOf(ctx).Elem()
 
-	if ctxType.Kind() == reflect.Struct {
-		for i := 0; i < ctxVals.NumField(); i++ {
-			currField, currIf := extractField(ctxVals, ctxType, i)
-			switch currField {
-			case "Context":
-				getKeyValue(currIf, m)
-			case "key":
-				nextField, nextIf := extractField(ctxVals, ctxType, i+1)
-				if nextField == "val" {
-					m[currIf] = nextIf
-					i++
+		if ctxType.Kind() == reflect.Struct {
+			for i := 0; i < ctxVals.NumField(); i++ {
+				currField, currIf := extractField(ctxVals, ctxType, i)
+				switch currField {
+				case "Context":
+					getKeyValue(currIf, m)
+				case "key":
+					nextField, nextIf := extractField(ctxVals, ctxType, i+1)
+					if nextField == "val" {
+						m[currIf] = nextIf
+						i++
+					}
 				}
 			}
 		}