Tn/swap mux for chi #843

TylerNoblett · 2023-05-26T14:37:13Z

After doing a bunch of research, it seemed that Chi was the best alternative to gorilla/mux, so I've swapped them out in this PR.

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

backend/authschemes/localauth/local_auth.go

backend/server/dissectors/dissected_request.go

backend/server/dissectors/dissected_request_test.go

backend/Readme.md

backend/server/api.go

backend/server/middleware/authenticator.go

backend/server/remux/routing_wrappers.go

backend/server/dissectors/dissected_request.go

Co-authored-by: Joel Smith <joel.smith@originate.com>

TylerNoblett · 2023-06-05T19:14:15Z

@JoelAtDeluxe I think I've addressed all of your feedback - thanks again for taking a look!

backend/bin/dev/dev.go

TylerNoblett added 6 commits May 25, 2023 14:30

remove most mux references

730e3b4

remove unneeded wrappers

3d7041e

remove extraneous middleware

ecddbd7

update gorilla references in readme/comments

5d75d10

fix metric routes

83ec059

remove last go mux reference

823dfc7

TylerNoblett requested review from jkennedyvz, jrozner and JoelAtDeluxe as code owners May 26, 2023 14:37

TylerNoblett commented May 26, 2023

View reviewed changes

backend/authschemes/localauth/local_auth.go Outdated Show resolved Hide resolved

TylerNoblett commented May 26, 2023

View reviewed changes

backend/server/dissectors/dissected_request.go Outdated Show resolved Hide resolved

TylerNoblett commented May 26, 2023

View reviewed changes

backend/server/dissectors/dissected_request_test.go Outdated Show resolved Hide resolved