utils: add Go 1.20+ way of converting string to byte slice (#2462)

* utils: add Go 1.20+ way of converting string to byte slice

Ref. https://github.com/valyala/fasthttp/blob/d2f97fc426ed451e64dc8e35e7f87a1d4a2d7bde/s2b_old.go.
Ref. https://github.com/valyala/fasthttp/blob/d2f97fc426ed451e64dc8e35e7f87a1d4a2d7bde/s2b_new.go.

* utils: fix golangci-lint apparently running with Go < 1.20

See https://github.com/gofiber/fiber/actions/runs/4968641325/jobs/8891360463?pr=2462.

utils/convert.go

@@ -13,29 +13,13 @@ import (
 	"unsafe"
 )
 
-const MaxStringLen = 0x7fff0000 // Maximum string length for UnsafeBytes. (decimal: 2147418112)
-
 // UnsafeString returns a string pointer without allocation
 //
 //nolint:gosec // unsafe is used for better performance here
 func UnsafeString(b []byte) string {
 	return *(*string)(unsafe.Pointer(&b))
 }
 
-// UnsafeBytes returns a byte pointer without allocation.
-// String length shouldn't be more than 2147418112.
-//
-//nolint:gosec // unsafe is used for better performance here
-func UnsafeBytes(s string) []byte {
-	if s == "" {
-		return nil
-	}
-
-	return (*[MaxStringLen]byte)(unsafe.Pointer(
-		(*reflect.StringHeader)(unsafe.Pointer(&s)).Data),
-	)[:len(s):len(s)]
-}
-
 // CopyString copies a string to make it immutable
 func CopyString(s string) string {
 	return string(UnsafeBytes(s))

utils/convert_s2b_new.go

@@ -0,0 +1,13 @@
+//go:build go1.20
+// +build go1.20
+
+package utils
+
+import (
+	"unsafe"
+)
+
+// UnsafeBytes returns a byte pointer without allocation.
+func UnsafeBytes(s string) []byte {
+	return unsafe.Slice(unsafe.StringData(s), len(s))
+}

utils/convert_s2b_old.go

@@ -0,0 +1,25 @@
+//go:build !go1.20
+// +build !go1.20
+
+package utils
+
+import (
+	"reflect"
+	"unsafe"
+)
+
+const MaxStringLen = 0x7fff0000 // Maximum string length for UnsafeBytes. (decimal: 2147418112)
+
+// UnsafeBytes returns a byte pointer without allocation.
+// String length shouldn't be more than 2147418112.
+//
+//nolint:gosec // unsafe is used for better performance here
+func UnsafeBytes(s string) []byte {
+	if s == "" {
+		return nil
+	}
+
+	return (*[MaxStringLen]byte)(unsafe.Pointer(
+		(*reflect.StringHeader)(unsafe.Pointer(&s)).Data),
+	)[:len(s):len(s)]
+}