-
Notifications
You must be signed in to change notification settings - Fork 70
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
faro-web-sdk API calls are going in loop #663
Comments
hi @snehalk-mdsol - @codecapitano is on PTO right now, but can you provide some more details regarding the issue you are facing? I am not fully understanding what the problem is. |
Hi @snehalk-mdsol I can't access the PR you linked. |
I have a suspicion. We'll release it soon. Unfortunately I couldn't reproduce the issue so once released please let us know if the update fixed the problem. |
We are seeing this as well. It is a big problem causing lots of failed logs. |
@Shawns2759 do you use the same setup with the |
|
Things are set up very similarly to this PR. |
Architecture is a monolith not a micro frontend right?
|
Our application is a Capacitor application that is serving a nextJs frontend and a a mobile frontend for Android and Apple via Capacitor. We delineate between mobile and web using We are using Grafana Cloud. Faro is still at 1.5.1. |
Thank you @Shawns2759 With Faro 1.8.0 we fixed an issue where session timings could drift between the faro receiver and faro itself which lead to
|
Interesting. Yes I will try that. |
@codecapitano I updated to
|
Can you share a screenshot with the request response information, like request/response headers etc. |
Sharing here in case it is useful for debugging and maybe a possible temporary fix for people facing it. From our testing we came up with the theory that the issue comes from the sheer amount of requests that can be generated by the SDK which apparently causes a race condition somewhere and causes preflight requests to not be performed, hence CORS fails and this error is shown. Attached a picture of how it looks in network tab To solve it we did 2 things. First was batching with a higher timeout and on top of that increased the default batching item limit to 150 since the default is 50
Again this is just a theory for now and i don't know if it's the same problem for everyone i'm just starting to monitor the result to see if we still continue seeing this issues. If there's a different issue or way to solve it would be very interested to know 🙏 Note: As an additional note here i'd like to mention that we actually see a very similar behaviour in Datadog which we are migrating away from. Some requests are marked as pending as well |
@codecapitano We have also found similar issue in Sentry and seems to be related to keepalive in fetch getsentry/sentry-javascript#7546 maybe this should be revisited in faro as well. Since we have a fork of faro due to circumstances we did the change and are monitoring to see if we still get errors as the comment above around limits and batching did not fix the error |
Facing a faro issue from quite a long now.
Please find below screenshots -
I tried updating faro libraries to the latest version 1.9.0 and added collector URLs in ignoreURLs while initializing a faro instance.
You can check the PR here - https://github.com/mdsol/platform-portal/pull/619/files
Can someone please help.
@codecapitano
The text was updated successfully, but these errors were encountered: