dba611db24
Some clients calling Canvas are including their own sentry-trace header, which is a header designed to associate traces from multiple services together; if that header is present, Sentry decides that the transaction should be sampled to maintain the association. This change discards the `sentry-trace` header unless there is also a `Referrer` header which contains the same host the request is for. This change also switches to using a proc to define the APM sample rate, which will allow the sample rate to be updated without restarting app servers. flag=none closes DE-1029, DE-1028 test plan: - verify that tracing can still be enabled/disabled by modifying the sampling rate - verify that calling Canvas with a `sentry-trace` header does not override the sampling rate, unless a same-origin `Referrer` header is also present Change-Id: Iddc80c3234975db0e45fa8defcba8eda1f4e0d24 Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/284485 Reviewed-by: Ryan Norton <rnorton@instructure.com> Product-Review: Ryan Norton <rnorton@instructure.com> QA-Review: Isaac Moore <isaac.moore@instructure.com> Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com> |
||
|---|---|---|
| .. | ||
| request_throttle | ||
| load_account.rb | ||
| prevent_non_multipart_parse.rb | ||
| request_context_generator.rb | ||
| request_context_session.rb | ||
| request_throttle.rb | ||
| samesite_transition_cookie_store.rb | ||
| sentry_trace_scrubber.rb | ||
| sessions_timeout.rb | ||