nil pointer dereference in internal/net/http/transport

[rinx]

Describe the bug:

application crashes when res.StatusCode is nil.

vald/internal/net/http/transport/roundtrip.go

Line 72 in e8ce139

switch res.StatusCode {

currently, this line is used only in agent-sidecar.

To Reproduce:

start agent-sidecar with invalid URL for S3 endpoint.

Expected behavior:

don't crash.

Environment:

• Go Version: 1.14.3
• Docker Version: 19.03.8
• Kubernetes Version: 1.18.2
• NGT Version: 1.11.5

[issue-label-bot]

Issue-Label Bot is automatically applying the label type/bug to this issue, with a confidence of 0.96. Please mark this comment with 👍 or 👎 to give our bot feedback!

Links: app homepage, dashboard and code for this bot.