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.

Sign up for GitHub

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

Jump to bottom

fix: web otel traces should not include the port by default #4366

Merged

sprutton1 merged 1 commit into main from otel_fix

Aug 19, 2024

Contributor

sprutton1 commented Aug 19, 2024

window.location.host includes the port, so we don't need to specify it here. It will be set by env vars at build time otherwise


          fix: web otel traces should not include the port by default

5c20684

github-actions bot added the A-web label

sprutton1 enabled auto-merge

August 19, 2024 21:23

sprutton1 added this pull request to the merge queue

Merged via the queue into main with commit 995d13f

8 checks passed

sprutton1 deleted the otel_fix branch

August 19, 2024 21:28

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels