Closed dwisiswant0 closed 1 year ago
Patch coverage has no change and project coverage change: +0.07
:tada:
Comparison is base (
3bc6527
) 71.70% compared to head (60b537b
) 71.78%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
IMPORTANT: Please do not create a PR without creating an issue first!
(Any change needs to be discussed before proceeding. Failure to do so may result in the rejection of the pull request).
Summary
This commit fixes a bug in the
checkBadReferrer
method where a nil pointer dereference occurred. The code was checking if an error occurred and returning it without handling the case when the error is nil. This caused a panic at runtime when trying to access a nil pointer. The fix removes the error logging statement and returns early when the error is nil, preventing the panic from happening.Proposed of changes
This PR fixes/implements the following bugs/features:
How has this been tested?
Proof:
Closing issues
Fixes #60
Checklist:
make lint
).