-
Notifications
You must be signed in to change notification settings - Fork 9
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
update deps, add edge support #15
Conversation
agadzik
commented
Aug 23, 2023
- Updating dependencies to the latest versions
- Adds Edge support for OTEL tracing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Orca Security Scan Summary
Status | Check | Issues by priority | |
---|---|---|---|
Passed | Secrets | 0 0 0 0 | View in Orca |
method: "POST", | ||
body: JSON.stringify(serviceRequest), | ||
headers: { | ||
"Content-Type": "application/json", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
without additional headers from OTEL_EXPORTER_OTLP_HEADERS
this won't do much for folks.
I've copied this into our codebase to test out (including fixing the configuration) but was not able to tickle it to send spans to my collector. |
@DavidS-ovm yes. We are doing a major review on this package, so this would likely change significantly. Should have some additional info on this soon. |