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

Jump to bottom

Update for HTTPS and New URL #15

Open
clintre opened this issue Jul 6, 2015 · 1 comment
Open

Update for HTTPS and New URL #15

clintre opened this issue Jul 6, 2015 · 1 comment

Comments

@clintre
Copy link

clintre commented Jul 6, 2015

I was using this on a project that requires SSL and when I used gChart it came back with security issues.

In looking into it I noticed that this used the old API path chart.apis.google.com which can't use HTTPS.

So I replaced that with the newer url and changed the full url calls to https and it worked well so far. The example page all seemed to work fine. I am sure there are some things that need to be added to take advantage of the API fully. However it is a quick fix if anyone needs to use it with the HTTPS requirement.
@pacbard
Copy link
Owner

pacbard commented Jul 9, 2015

That would be a great addition to the project. Do you mind opening a pull request with the API address change?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pacbard @clintre