-
Notifications
You must be signed in to change notification settings - Fork 151
Does this SDK works with browser extension ? #487
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
Comments
Hi @bt0r, We are not actively supporting browser extension so some features could not work. |
Hi @bcaudan Thanks for your reply :) |
If it does not bother you, it would be great |
I pushed a branch with a litle test https://github.com/bt0r/NoteFlix/tree/draft/datadog-logs
Index.js is the extension entrypoint, i simply added https://github.com/bt0r/NoteFlix/blob/draft/datadog-logs/src/index.js#L3-L20 to trigger the error (readable in console) If i can help you, just let me know ! i would be very happy to use datadog for this project |
I've made a few tests, it works fine on chrome but it fails on firefox. On firefox, I've got a |
Unless the same request works from a different context (curl, browser debug console, integration in regular web page), it seems to me that this is a different issue not directly related to this repository. If this is the case, you should ask Datadog support for help with it. |
The bug is related to the "datacenter" attribute chosen in the init section https://github.com/bt0r/NoteFlix/blob/draft/datadog-logs/src/index.js#L11, i chose "eu" and now with "us" it's working fine, i close this issue. Thanks you for you quick answers and help ! |
Hi there 👋🏻
In advance, thanks for your work 👏🏻
I'm actually trying to add your sdk inside a browser extension (in a content_script) but i stuck on
'logs.logger.info' not yet available, please call '.init()' first.
Thanks
The text was updated successfully, but these errors were encountered: