Decrease log level for 4xx HTTP errors

[ccoVeille]

Pull Request Template

Description:

• Closes issue Avoid Logging 4XX Status Codes as Errors #1279
• Replaces Fix/4xx logs #1295

Checklist:

• I have formatted my code using goimport and golangci-lint.
• All new code is covered by unit tests.
• This PR does not decrease the overall code coverage.
• I have reviewed the code comments and documentation for clarity.

Thank you for your contribution!

[coolwednesday]

@ccoVeille, a better name for the PR would be appreciated.

[Umang01-hash]

@ccoVeille I tested your solution and it is working perfectly fine.
I have updated the description.
Please also update the PR name.

And also it would be nice if you can add test for the newly added code.

[ccoVeille]

This PR target branch was your PR one @Umang01-hash

• Fix/4xx logs #1295

It explains why the PR description was so basic. The tests were the ones you added.

I should have mentioned and use draft.

I was planning you to merge them in your branch, then keep the discussion running in your PR.

I'll sort that, no worries.

[ccoVeille]

I implemented here what I initially planned.

But I felt like it was not a good way to do. Something that couldn't scale easily,

So I provided an alternative implementation that would allow more than the current one.

See

• Decrease log level for 4xx HTTP errors : alternative option #1304

For me, it supersedes the current PR, that now becomes pointless.