-
-
Notifications
You must be signed in to change notification settings - Fork 323
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
Duplicated HTTP spans and breadcrumbs #4365
Comments
@dsychev80 thanks for the report, we're gonna investigate and follow up here |
Thank you for your report @dsychev80 🙇 |
Hello @antonis!
Hope this helps |
Thank you for your response and the information provided @dsychev80 |
Hello @dsychev80 👋 |
Hello @antonis
|
Hey @dsychev80, I have a few questions, which I will number through this issue so it's easier for you to reply. Your logs show failed requests.
If not, maybe we need to check
We safeguard adding duplicated breadcrumbs here, so I'm a bit surprised that we see duplicates sentry-cocoa/Sources/Sentry/SentryNetworkTracker.m Lines 485 to 490 in 33c23d3
Also, the URL in the error message points to some type of Sentry:
The SDK should ignore requests to your DSN. It could be something is not working correctly. sentry-cocoa/Sources/Sentry/SentryNetworkTracker.m Lines 159 to 163 in 33c23d3
And finally,
|
Hello @philipphofmann Answers for your questions:
|
Thank you for the extra information @dsychev80 🙇 I was wondering if you could experiment with using another api url to understand if the error is triggered by the specific server configuration or not. For example could you hardcode some public api url (e.g. |
This is an interesting idea @antonis , and i'll try it later |
Thank you @dsychev80 🙇
Regarding the
|
Hi guys! @antonis, first of all, sorry for the delay 🤗. Today, I tried to investigate the issue following your advice, and here’s what I found: I placed a simple network call in viewDidLoad (implementation attached in screenshots), but the duplication issue is still there. As you mentioned above regarding the suspended state, in the notes from the URLSessionTask of the resume method, we can see that the task begins in that state, so maybe this is a hint🤔. |
Thank you for testing this @dsychev80 and providing the results 🙇 |
Originally posted by @dsychev80 in #1409
The text was updated successfully, but these errors were encountered: