-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: AWS xray tracing config #44
Conversation
If you run the daemon locally you can see data in the AWS console. |
Xray daemon config is now in the handbook |
Juicy errors now!
|
Kudos, SonarCloud Quality Gate passed! |
@Judge40 I am at a bit of an impasse here, wondering what you think. The 'segment cannot be found' errors seem to suggest the servlet filter is not properly configured (as per 1st point on https://docs.aws.amazon.com/xray/latest/devguide/xray-troubleshooting.html). The latest version of the Xray Recorder SDK still implements a |
Related issue: aws/aws-xray-sdk-java#364 Looks like it's fix/merged but not yet released. |
Ok cool, thanks for confirming @Judge40 . Given we are not going 'live' by any realistic measure for the next couple of months at least, shall we just park this piece of work and pick it up when Xray catches up with us? :) |
fcbd7e5
to
9ee8a62
Compare
Update the AWS Xray SDK to `2.14.0`, which allows it to support Spring 6's Jakarta based Filters. Move the tracing filter configuration to FilterConfiguration as the tracing filter must be placed before the SignedDataFilter in the filter chain. Failure to register them in this order will cause the trace to fail for any unsigned data, as the SignedDataFilter returns an error instead of continuing the filter chain calls. TIS21-4226
TIS21-4226
TIS21-4226
Kudos, SonarCloud Quality Gate passed! |
TIS21-4226