search

electrode-io / electrode

Web applications with node.js and React
Other
2.1k stars 301 forks source link

chore: replace `polyfill.io` #2006

Closed SukkaW closed 2 months ago

SukkaW commented 6 months ago

electrode loads polyfill from the domain polyfill.io. The PR replaces polyfill.io with cdnjs.cloudflare.com/polyfill.

polyfill.io was acquired by a China-based CDN company "Funnull", see the announcement from the polyfill.io domain owner's Twitter and https://github.com/polyfillpolyfill/polyfill-service/issues/2834. Despite Funnull's claims of operating in the United States, the predominance of Simplified Chinese on its website suggests otherwise, and it turns out that "Funnull" is notorious for providing service for the betting and pornography industries.

The original creator of the polyfill.io has voiced his concern on Twitter. And since the acquisition, numerous issues have emerged (https://github.com/polyfillpolyfill/polyfill-service/issues/2835, https://github.com/polyfillpolyfill/polyfill-service/issues/2838, https://github.com/alist-org/alist/issues/6100), rendering the polyfill.io service extremely unstable. Since then, Fastly (Announcement) and Cloudflare (Announcement) has hosted their own instances of polyfill.io service.

CLAassistant commented 6 months ago

CLA assistant check
All committers have signed the CLA.

arunvishnun commented 6 months ago

Thank you @SukkaW, appreciate your contribution. We will review this.

arunvishnun commented 2 months ago

Hi @SukkaW To publish the changes, we needed change files generated by rush. So we included your commit to this PR https://github.com/electrode-io/electrode/pull/2018 and published latest version for this and packages that are dependent on this. Changes logs here. With this we will close this PR.

Thanks again for your contribution!