search

pacbard / gChartPhp

Apache License 2.0
88 stars 45 forks source link

Update for HTTPS and New URL #15

Open clintre opened 9 years ago

clintre commented 9 years ago

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 commented 9 years ago

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