Skip to content

Add function to extract sentry trace info from traceparent string.#1771

Closed
antonpirker wants to merge 1 commit into
antonpirker/1687-basic-otel-support-2-instrumenterfrom
antonpirker/1687-basic-otel-support-3c-extract-sentry-trace
Closed

Add function to extract sentry trace info from traceparent string.#1771
antonpirker wants to merge 1 commit into
antonpirker/1687-basic-otel-support-2-instrumenterfrom
antonpirker/1687-basic-otel-support-3c-extract-sentry-trace

Conversation

@antonpirker
Copy link
Copy Markdown
Contributor

This is basically the opposite of to_traceparent().

@antonpirker antonpirker added this to the Basic OTel support milestone Nov 30, 2022
@sl0thentr0py sl0thentr0py changed the base branch from master to antonpirker/1687-basic-otel-support-3b-timestamps November 30, 2022 13:59
@sl0thentr0py
Copy link
Copy Markdown
Member

sl0thentr0py commented Nov 30, 2022

we already have this in python, do we really need this?

def extract_sentrytrace_data(header):

plus this is not using the regex so it's a less precise implementation to just split on the dashes

Base automatically changed from antonpirker/1687-basic-otel-support-3b-timestamps to antonpirker/1687-basic-otel-support-2-instrumenter November 30, 2022 15:54
@antonpirker
Copy link
Copy Markdown
Contributor Author

Oh, good to know! Will close and use the existing thing!

@antonpirker antonpirker closed this Dec 1, 2022
@antonpirker antonpirker deleted the antonpirker/1687-basic-otel-support-3c-extract-sentry-trace branch April 19, 2023 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants