enable passing custom logger

[LukasJenicek]

This way I can let ppl to modify the logging in any way..

We could have in our codebase slog.Logger like this:

type Slogger struct {
	log                  *slog.Logger
	printResponsePayload bool
}

func NewSlogger(log *slog.Logger, printResponsePayload bool) *Slogger {
	return &Slogger{
		log:                  log,
		printResponsePayload: printResponsePayload,
	}
}

func (s *Slogger) LogRequest(r *http.Request, curl *http2curl.CurlCommand) {
	s.log.Info(curl.String())
	s.log.Info(fmt.Sprintf("request: %s %s", r.Method, r.URL))
}

func (s *Slogger) LogResponse(r *http.Request, resp *http.Response, startTime time.Time) {
	if resp == nil {
		s.log.Warn(fmt.Sprintf("response (<nil>): %v", r.URL.String()), slog.Duration("duration", time.Since(startTime)))
		return
	}

	if resp.StatusCode >= 400 {
		s.log.Error(fmt.Sprintf("response: %s %s", resp.Status, resp.Request.URL))
		return
	}

	s.log.Info(fmt.Sprintf("response: %s %s", resp.Status, resp.Request.URL))

	if s.printResponsePayload && resp.Header.Get("Content-Type") == "application/json" {
		var b bytes.Buffer

		tee := io.TeeReader(resp.Body, &b)
		resp.Body = io.NopCloser(&b)

		payload, err := io.ReadAll(tee)
		if err == nil {
			// Pretty print the JSON payload
			var prettyJSON bytes.Buffer
			if err := json.Indent(&prettyJSON, payload, "", "    "); err == nil {
				s.log.Info(prettyJSON.String())
			}
		}
	}
}

[VojtechVitek]

I'm not a fan of http2curl being part of any public interface

[LukasJenicek]

Makes sense. Should I pass string directly ?