⚡ set raw to nil in JSON to avoid memory leak

[damianham]

Description

1 line change to set raw to nil in c.JSON() to avoid memory leak

Fixes #2470

Type of change

• Bug fix (non-breaking change which fixes an issue)

Checklist:

• 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

[gaby]

@damianham Can you take a look at the lint error

[ReneWerner87]

are you sure this solves the memory problem?
have you already tested this in use?

[damianham]

No it doesn't

[shohamroditimemphis]

Hi,
I am experiencing memory leak when I am using with c.Params().
This issue with memory leak was solved in the latest version?

Thanks

[ReneWerner87]

Hi,
I am experiencing memory leak when I am using with c.Params().
This issue with memory leak was solved in the latest version?

Thanks

There was no memory leak

Pls read the hint box
https://docs.gofiber.io/next/api/ctx#params

[damianham]

Hi, I am experiencing memory leak when I am using with c.Params(). This issue with memory leak was solved in the latest version?

Thanks

The memory leak I was experiencing was in in the JSON encoder. I solved it by changing all instances of
return c.JSON(fiber.Map{"result": business})
to
`
res := fiber.Map{"result": business}

outp, _ := json.Marshal(res)

c.Response().Header.SetContentType("application/json")

c.SendString(string(outp))

return nil

`