Skip to content
This repository has been archived by the owner on Sep 27, 2022. It is now read-only.

Add crossorigin="anonymous" #31

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

leahciMic
Copy link

@leahciMic leahciMic commented Dec 14, 2016

The OpenTok JS SDK has experimental error reporting. For this to work well across domains, we need to do the following:

  • Ensure the OpenTok JS SDK is served with Access-Control-Allow-Origin: *
  • Ensure the script is loaded with the attribute crossorigin="anonymous"

The OpenTok JS SDK has experimental error reporting. For this to work well
across domains, we need to do the following:

* Ensure the OpenTok JS SDK is served with `Access-Control-Allow-Origin: *`
* Ensure the script is loaded with the attribute `crossorigin="anonymous"`
@aullman
Copy link

aullman commented Dec 14, 2016

@leahciMic is this something that we need to tell all of our partners to do?

@leahciMic
Copy link
Author

@aullman yes, I believe so, otherwise, we will not get any error information.

@leahciMic
Copy link
Author

@aullman there are alternatives. Hopefully, there are much better options but the ones that come to mind atm are:

  • Wrapping our library's methods, functions and code blocks in try/catch and sending the errors off.
  • Automating the first option.

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

Successfully merging this pull request may close these issues.

2 participants