pkg/receive: return correct status code

[squat]

status code, i.e. it does not overwrite the status code returned by the
downstream handler.

Signed-off-by: Lucas Servén Marín lserven@gmail.com

cc @metalmatze @brancz @kakkoyun @bwplotka

[metalmatze]

Did you check what happens now if we actually have a gateway timeout?

[squat]

If there's a gateway timeout, then err is not nil, so we hit this line of code:

telemeter/pkg/receive/handler.go

Line 63 in 9562123

http.Error(w, err.Error(), http.StatusBadGateway)

[brancz]

/lgtm

[brancz]

/retest

[bwplotka]

👍

/lgtm

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: brancz, bwplotka, squat

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [brancz,bwplotka,squat]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[squat]

/retest

[bwplotka]

As a side note this is something we can improve: Ideally we should use something like httputil.ReverseProxy which would handle all those things automatically.