search

ireade / caniuse-embed

📊 Embed realtime data from caniuse.com
https://caniuse.bitsofco.de
325 stars 21 forks source link

Always use https if available #6

Closed Schweinepriester closed 6 years ago

Schweinepriester commented 7 years ago

Thanks for creating this! :)

Just a little change to encourage HTTPS. Per https://www.paulirish.com/2010/the-protocol-relative-url/:

Now that SSL is encouraged for everyone and doesn’t have performance concerns, this technique is now an anti-pattern. If the asset you need is available on SSL, then always use the https:// asset.

http://cdn.jsdelivr.net/caniuse-embed/1.1.0/caniuse-embed.min.js seems to be redirecting to HTTPS anyway, but this way people embed this the 'more correct' way.

ireade commented 6 years ago

Sorry this took so long to get to, thanks for doing this. Merging now :)