fix: fix log nil error when http request failed #314

nanjingfm · 2022-11-11T11:42:20Z

Changes

As the author of this PR, please check off the items in this checklist:

spec PR link included
Follows the commit message standard
Meets the contributing guidelines (including
functionality, content, code)
Test cases with documentation and functionality works as expected using current and related github repos (MUST deploy and check)
Release notes block below has been filled in or deleted (only if no user facing changes)

errors/errorhandling.go

alaudabot · 2022-11-11T11:53:56Z

_{updated at Mon Nov 21 02:53:19 UTC 2022}

Status: OK 🟢

Metric	Change set	Target _(main)	After
🧪 Code coverage	0.00% _{Coverage on 2 new lines}	53.90%	53.90% ↔️ 🟢 _{estimated after merge} _{allowed difference 0}
👥 Duplications	0.00% _{Duplications on 0 new lines}	0.20%	0.20% ↔️ 🟢 _{estimated after merge} _{allowed difference 0}

[314]

nanjingfm commented Nov 11, 2022

View reviewed changes

errors/errorhandling.go Outdated Show resolved Hide resolved

nanjingfm force-pushed the fix/nil-log-message branch 2 times, most recently from d322524 to 6532a99 Compare November 11, 2022 12:32

nanjingfm force-pushed the fix/nil-log-message branch from 6532a99 to 56c7940 Compare November 21, 2022 02:28

fix: fix log nil error when http request failed

e8725e3

nanjingfm force-pushed the fix/nil-log-message branch from 56c7940 to e8725e3 Compare November 21, 2022 02:44

airycanon approved these changes Nov 21, 2022

View reviewed changes

nanjingfm merged commit 3b738bb into main Nov 21, 2022

nanjingfm deleted the fix/nil-log-message branch November 21, 2022 03:45