🩹Fix: Startup message doesn't show correct amount of Handlers after mounting

[capthiron]

After mounting an App instance or Group to an existing instance (root), the handlerCount of the root is not being updated. I came across this issue when I noticed that the startup message showed Handlers=0 even though my app had mounted a router with multiple handlers.

For example...

router := New()
router.Get("/doe", func(c *Ctx) error {
    return c.SendStatus(StatusOK)
})

app := New()
app.Mount("/john", router)

... currently results in the following startup message.

You get the same result when mounting the router to a Group.

I fixed this by simply adding the handlerCount of the mounted instance to the root App's own handlerCount in the corresponding Mount-function of the App and Group struct.

Side note
When writing the tests I noticed that when registering a Get-Route, not 1 but 2 Handlers are being registered. This is not the case for all the other HTTP-Methods. After investigating I noticed that App.Get() not only adds the Get-Method but also the Head-Method which results in the second handler. I'm wondering whether that is this the desired behaviour or not?

Best regard, Dario ⚡

[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]

thanks, its looking good

[welcome]

Congrats on merging your first pull request! 🎉 We here at Fiber are proud of you! If you need help or want to chat with us, join us on Discord https://gofiber.io/discord