chore: add security logs to webhook verification failures

[notfromstatefarm]

Signed-off-by: notfromstatefarm 86763948+notfromstatefarm@users.noreply.github.com

[codecov]

Codecov Report

Merging #10372 (4052f56) into master (68d0ef0) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master   #10372      +/-   ##
==========================================
- Coverage   46.25%   46.23%   -0.02%     
==========================================
  Files         228      228              
  Lines       27883    27893      +10     
==========================================
  Hits        12897    12897              
- Misses      13247    13253       +6     
- Partials     1739     1743       +4     

Impacted Files	Coverage Δ
util/webhook/webhook.go	65.27% <0.00%> (-3.17%)	⬇️
util/settings/settings.go	51.36% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[crenshaw-dev]

Would it be terribly difficult to pull the payload parsing/checking logic into a function and unit test it (maybe even unit testing that the logs happen)?

Not sure how much work it would be to mock the payloads that would produce these errors.

[crenshaw-dev]

Merging without tests. At worst it's either no-logs or excessive logs. Either one is easy enough to deal with later. Thanks @notfromstatefarm!