Open jaedle opened 3 months ago
I have already seen #5554 and hope that @wangzlei could jump in reviewing the PR I would love to raise. :pray:
@jaedle Is this issue still need to fix or already solved?
@jaedle Is this issue still need to fix or already solved?
Still open, see #5718.
@jaedle Thanks , I am starting working on it.
@ayushrakesh I am sorry if you got this wrong, but did you have a look at the existing PR #5718?
@jaedle Oh, there is already a PR. Sorry.
Ok 👌
May I ask why you created a new PR?
Having a look at your changes those look pretty similar to the existing PR which was already reviewed.
Yeah, sorry for it, I thought PR is not made yet.
I am very sorry if I was unclear 🙁
No no, that's my fault.🥲
Description
The X-Ray remote sampler may silently fail obtaining the sampling rules from the awsproxy.
We are running the OpenTelemetry Collector behind a reverse proxy as we have about a dozen services ingesting traces which are forwarded to AWS X-Ray. As this adds forward headers (i.e. x-forwarded-*), the request signing of the awsproxy was not working correctly.
Even though the response of AWS results in a client error, there are no signs of an error. The JSON response body is unmarshaled and treated as empty ruleset.
The missing check on status code of the response should be the problem in this case.
The only response code for a successful request should be 200/OK according the documentation.
Environment
Should not be relevant for the error.
Steps To Reproduce
The simplest way to reproduce may be a test case for
client_test.go
:Current behavior
Expected behavior
I would love to provide a PR to fix that issue.