Skip to content
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

Ad blockers #1078

Closed
yaronvhop opened this issue Sep 10, 2021 · 6 comments
Closed

Ad blockers #1078

yaronvhop opened this issue Sep 10, 2021 · 6 comments

Comments

@yaronvhop
Copy link

yaronvhop commented Sep 10, 2021

it seems that APM RUM is blocked by ublock origin ad blocker.

The rule:

easylist/easylist@e0840e0

Any way to get pass it? Might be a option a to have that we can change the URL to avoid those ad blockers blocking performance data?

Thanks

@gadirotenberg
Copy link

It seems that the rum/events suffix of the URL is inlined in the code:

const endPoint = cfg.get('serverUrl') + `/intake/v${apiVersion}/rum/events`

It will be great to have it as a parameter. On the server side, a possible usage for users that deploy APM using k8s, can be setting an ingress rule to map a suffix of choice to rum/events.

@alexbode
Copy link
Contributor

I am seeing the same problem of being blocked in the firefox and brave browser. Anyway to add for example serverEndpoint to the Config and have the

const endPoint = cfg.get('serverUrl') + cfg.get('serverEndpoint')

like @gadirotenberg mentioned?
Or how can someone else make that changes?

@vigneshshanmugam
Copy link
Member

Anyone wants to do a PR to allow customizing the serverUrlPrefix option? It should be fairly small change.

@alexbode
Copy link
Contributor

I can make the PR

alexbode pushed a commit to alexbode/apm-agent-rum-js that referenced this issue Sep 28, 2021
Change hardcoded serverUrlPrefix to passable parameter to evade ad blockers

fixes issue elastic#1078
alexbode pushed a commit to alexbode/apm-agent-rum-js that referenced this issue Sep 29, 2021
* add serverUrlPrefix documentation to configuration.asciidoc
* add unittest 'should update serverUrlPrefix'

fixes issue elastic#1078
@nicooprat
Copy link

Looks like Brave now blocks it by default too, without any extension installed. Looking forward to this configuration!

@vigneshshanmugam
Copy link
Member

fixed by #1081

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants