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.

Sign up for GitHub

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

Tainted canvas error fix #37

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Tainted canvas error fix #37

wants to merge 2 commits into from

Commits on May 25, 2016

  1. fixed the CORS issue which may cause tainted canvas error 

    when loading the remote images we might encounter the tainted canvas error, I added the cross origin policy to overcome this situation. note that the remote server need to send proper access control headers in order this to work which means the remote server should allow using their images publicly or you can write a proxy to send your headers which is not legally right.
    @shivenigma
    shivenigma committed May 25, 2016
    Configuration menu
    Copy the full SHA
    de0fa3c View commit details
    Browse the repository at this point in the history

  2. minified version of the CORS error fix

    @shivenigma
    shivenigma committed May 25, 2016
    Configuration menu
    Copy the full SHA
    ef1e94e View commit details
    Browse the repository at this point in the history