fix related to concurrent write to map issue

[dheerajv4855]

tested it locally for around two hours by frequently changing the channels
working good now , didn't see any crash after changes

[rabilrbl]

Hey @dheerajv4855, Thanks for the PR,

I wish you had seen my other PR migrating to gofiber #30 and waited for my reply in #31
gofiber is built on top of fasthttp, similar to gin built on top net/http, hence fasthttp is an alternative to net/http, So i migrated from net/http to fasthttp. fasthttp performs well on concurrency. But net/http is better with standards, which is not relevant in our small application.

Secondly, r.Run(":5001") requires firewall permission everytime you run on windows, because you are exposing the port to public and less secure. Hence I had put localhost:5001 and added optional argument to set host:port

Unfortunately, I won't be able to merge this PR.

Thanks again!

[dheerajv4855]

Yeah I don't want this pr to be merged I saw lot of work done to migrate to gofiber I hope that will solve the problem

…

On Sat, Sep 9, 2023, 21:31 Mohammed Rabil ***@***.***> wrote: Closed #35 <#35>. — Reply to this email directly, view it on GitHub <#35 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ALSHMC23WDEV25GABUO7Q2LXZSHENANCNFSM6AAAAAA4RO2IWY> . You are receiving this because you were mentioned.Message ID: ***@***.***>