-
Notifications
You must be signed in to change notification settings - Fork 44
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
UnhandledRejection error while loading the agent 1.235.0 #599
Comments
Thanks for reporting this issue @ilario-pierbattista. We will investigate why the chunk loading errors are not being caught. |
I encountered the same issue, any solution? |
Hi @patrickhousley, now it started to use the 1.236.0 version, and it started to fail with:
It seems a regression of #392 |
Warning logs or errors? I only see it as warnings which is working as intended. If it's a red error, which shouldn't be possible atm, then there's an issue. Silencing the warnings is not an option right now; as long as the agent is blocked, it will log those out. |
@cwli24 hi! Do you know when 1.236 will be available as a compiled JS snippet? |
@cwli24 nope, my bad, it was just a warning. |
I'm getting the same error. It started on Jun 30... |
Any progress with the |
I think the issue might've been found in the fetch keep-alive call on unload when it fails. We have an open bug for that specifically. |
thank you @cwli24 |
@matejbasic @ilario-pierbattista @tavito @matejbasic We recently released version 1.237.1 of the browser agent. Could you please let us know if this issue is still persisting? For APM injected apps, you may need to restart your APM instrumented applications. |
@patrickhousley it seems to be fixed, thanks |
@patrickhousley it seems to be fixed here too, thanks! |
Hello, We are still getting version 1.236.0, we've cleared cache, redeployed (ecs+docker), rebuilt images, rebuilt php-agent, nothing is working and we're still getting 1.236.0 Has this not been published yet or something? It's causing us a huge amount of issues because it flags on Sentry, so we're hoping to get this fix ASAP. |
@vekien Version 1.237.1 was published August 2nd. If you are using the PHP APM agent to inject the browser agent, you should only need to restart the application (which it seems you have done). Are you sure you are not using the copy/paste installation method? You may want to reach out to your New Relic support contact so the support team can further help. |
What is the copy/paste install method? This is how we have our agent installed (it's Docker)
And our application is still fetching 1.236.0, it's not cached either as we're getting thousands of Sentry's from over 100+ users and all our Devs are getting this same version. The Docker Tasks on ECS have been updated multiple times a day, so since the 2nd there has been well over 50+ new releases, so it has definitely restarted :D Does the PHP-Agent not pull the latest? Do we have to wait for that to update? |
@vekien You will want to reach out to the New Relic support team as this seems like a possible issue with the PHP APM agent. It should automatically get the newest version of the browser agent. We, the browser agent team, does not have any settings set right now to force this version. Sorry I could not help further. |
@patrickhousley no errors reported in a week, seems to be working fine now, thank you! 😊 |
Description
Starting from yesterday evening (2023-06-22 17:00 UTC), I'm getting the following error while loading the agent in browsers with strict privacy settings:
It's similar to #392. In the browser console I see chunks loading failures, and then the error
Uncaught (in promise)
.The agent is injected dinamically in HTML by the PHP agent.
Steps to Reproduce
Use a browser with strict privacy settings. It will block the requests to
https://js-agent.newrelic.com
.Expected Behavior
Even if the browser blocks the agent download I expect to get only warnings in console. Rejections should be handled.
Relevant Logs / Console output
Sample warnings:
Error:
Your Environment
Additional context
Similar issues #392
The text was updated successfully, but these errors were encountered: