feat(spanner): make randIDForProcess hexadecimal in x-goog-spanner-re…

…quest-id

Implements a new requirement that randIDForProcess should
be a hexadecimal value with the format specifier "%016x".

Fixes #11545

spanner/request_id_header.go

@@ -38,15 +38,15 @@ import (
 var randIDForProcess string
 
 func init() {
-	bigMaxInt64, _ := new(big.Int).SetString(fmt.Sprintf("%d", uint64(math.MaxUint64)), 10)
-	if g, w := bigMaxInt64.Uint64(), uint64(math.MaxUint64); g != w {
+	bigMaxUint64, _ := new(big.Int).SetString(fmt.Sprintf("%d", uint64(math.MaxUint64)), 10)
+	if g, w := bigMaxUint64.Uint64(), uint64(math.MaxUint64); g != w {
 		panic(fmt.Sprintf("mismatch in randIDForProcess.maxUint64:\n\tGot:  %d\n\tWant: %d", g, w))
 	}
-	r64, err := rand.Int(rand.Reader, bigMaxInt64)
+	r64, err := rand.Int(rand.Reader, bigMaxUint64)
 	if err != nil {
 		panic(err)
 	}
-	randIDForProcess = r64.String()
+	randIDForProcess = fmt.Sprintf("%016x", r64.Uint64())
 }
 
 // Please bump this version whenever this implementation

spanner/request_id_header_test.go

@@ -16,6 +16,7 @@ package spanner
 
 import (
 	"context"
+	"encoding/hex"
 	"encoding/json"
 	"fmt"
 	"math"
@@ -1925,3 +1926,13 @@ func TestRequestIDHeader_SingleUseReadOnly_ExecuteStreamingSql_UnavailableDuring
 		t.Fatalf("RequestID streaming segments mismatch: got - want +\n%s", diff)
 	}
 }
+
+func TestRequestID_randIDForProcessIsHexadecimal(t *testing.T) {
+	decoded, err := hex.DecodeString(randIDForProcess)
+	if err != nil {
+		t.Fatal(err)
+	}
+	if len(decoded) == 0 {
+		t.Fatal("Expected a non-empty decoded result")
+	}
+}