Trim spaces in forwarded header addresses

Open Dutchy- opened this issue 1 year ago • 2 comments

Fixes #81

I see you already made a fix for this in #82 but I've added a realistic test for multiple IPs in the headers.

Feel free to either merge this, copy the tests or close this PR.

Apr 15 '25 11:04 Dutchy-

Hi @Dutchy- Thanks for your contribution. I will probably merge it and keep the tests.

Apr 17 '25 06:04 PascalMinder

Would it be possible, that you fix the following? Then I will merge it.

golangci-lint run
Error: geoblock_test.go:139:18: string `Allowed request` has 3 occurrences, make it a constant (goconst)
	expectedBody := "Allowed request"

Apr 20 '25 09:04 PascalMinder

Sorry, I wasn't able to look at this for a few days but I've fixed it now :)

Apr 25 '25 11:04 Dutchy-