💄 feat(middleware/logger): ip match gin-gonic/gin formatter

[raulaguila]

Description

TagIP to match gin-gonic/gin defaultLogFormatter, specifically %15v instead of %v.

With %v:

| 127.0.0.1 |
| 192.168.1.10 |
| 105.103.142.227 |

With %15v, like gin https://github.com/gin-gonic/gin/blob/62b50cfbc0de877207ff74c160a23dff6394f563/logger.go#L143:

|       127.0.0.1 |
|    192.168.1.10 |
| 105.103.142.227 |

And, adds Test_Logger_LenTagIP.

Type of change

Checklist:

• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes

[welcome]

Thanks for opening this pull request! 🎉 Please check out our contributing guidelines. If you need help or want to chat with us, join us on Discord https://gofiber.io/discord

[ReneWerner87]

is it also working with ipv6 ?

[gaby]

@raulaguila See the comment above

[raulaguila]

Tested only with IPv4.

[gaby]

@raulaguila The unit-tests are failing here:

=== CONT  Test_Logger_All
    logger_test.go:206: 
        Test:       Test_Logger_All
        Trace:      logger_test.go:206
        Expect:     2226Host=example.comhttp0.0.0.0example.com/?foo=bar/Cannot GET /             (string)
        Result:     2226Host=example.comhttp        0.0.0.0example.com/?foo=bar/Cannot GET /     (string)

[ReneWerner87]

Tested only with IPv4.

Pls also consider this and add unittests for it