map-services / company-data-api / 26000731268

40%

main: 40%

Build Type

Pull #102

github

Committed by gemini-pr-agent[bot]

Commit Message

chore: iterative updates from Gemini based on PR comment

The `RequestLogger` middleware has been refactored to support variadic exclude paths and now mirrors the implementation pattern from the requested repository. Unit tests have been added to verify that the middleware correctly logs standard requests and skips excluded paths.

### Changes Summary
- **Refactored `internal/middleware/request.go`**: Updated `RequestLogger` to accept a logger and variadic `excludedPaths`. It now correctly filters requests based on the provided paths and uses the `slog` logger with consistent formatting.
- **Updated `cmd/api_server.go`**: Updated the middleware configuration to pass `slog.Default()` and the excluded paths (`/healthz` and `/metrics`) to `RequestLogger`.
- **Added `internal/middleware/request_test.go`**: Created new unit tests to confirm the logging behavior and path exclusion logic, ensuring they function as expected in `gin` environments.

Tests have been verified to pass using `go test`.

Pull Request Pull Request #102: fix: convert to JSON structured logging using `log/slog`

Coverage Stats

30 of 84 new or added lines in 11 files covered. (35.71%)

5 existing lines in 3 files now uncovered.

324 of 817 relevant lines covered (39.66%)

0.45 hits per line