all: change the signature of default request/response dumper

Instead of function that use the signature of HTTPRequestDumper/
HTTPResponseDumper; change it to function that _return_ HTTPRequestDumper/
HTTPResponseDumper.
In this way, the documentation can show the clear relation between
function and its type.

1 files changed, 10 insertions, 8 deletions

diff --git a/http_request_dumper.go b/http_request_dumper.go
index 9029693..6dd15bc 100644
--- a/http_request_dumper.go
+++ b/http_request_dumper.go
@@ -14,16 +14,18 @@ import (
 // [RunResponse] DumpRequest.
 type HTTPRequestDumper func(req *http.Request) ([]byte, error)
 
-// DumpHTTPRequest define default [HTTPRequestDumper] that convert
+// DefaultRequestDumper define default [HTTPRequestDumper] that convert
 // [http.Request] with its body to stream of bytes using
 // [httputil.DumpRequest].
 //
-// The returned dump have CRLF ("\r\n") replaced with single LF ("\n").
-func DumpHTTPRequest(req *http.Request) (raw []byte, err error) {
-	raw, err = httputil.DumpRequestOut(req, true)
-	if err != nil {
-		return nil, fmt.Errorf(`DumpHTTPRequest: %w`, err)
+// The returned bytes have CRLF ("\r\n") replaced with single LF ("\n").
+func DefaultRequestDumper() HTTPRequestDumper {
+	return func(req *http.Request) (raw []byte, err error) {
+		raw, err = httputil.DumpRequestOut(req, true)
+		if err != nil {
+			return nil, fmt.Errorf(`DefaultRequestDumper: %w`, err)
+		}
+		raw = bytes.ReplaceAll(raw, []byte{'\r', '\n'}, []byte{'\n'})
+		return raw, nil
 	}
-	raw = bytes.ReplaceAll(raw, []byte{'\r', '\n'}, []byte{'\n'})
-	return raw, nil
 }