Skip to content

security: remove polyfill.io script tag#351

Open
sinatragianpaolo wants to merge 1 commit intodevelopfrom
UN-2455
Open

security: remove polyfill.io script tag#351
sinatragianpaolo wants to merge 1 commit intodevelopfrom
UN-2455

Conversation

@sinatragianpaolo
Copy link
Collaborator

@sinatragianpaolo sinatragianpaolo commented Mar 9, 2026

Summary

  • Removes the polyfill.io CDN script tag from public/index.html

The polyfill.io domain was sold in 2024 to an external company that started injecting malicious JavaScript into CDN responses, targeting mobile users with redirects to scam sites. The original library author publicly recommended against using it, and AGID published an official advisory.

All loaded features (String.prototype.replaceAll, IntersectionObserver, Array.prototype.map, Array.prototype.reduce) are natively supported by all browsers released in the last 5 years — no alternative polyfill is needed.

Reference: https://www.agid.gov.it/it/notizie/polyfillio-il-cert-agid-consiglia-alle-pa-che-lo-utilizzano-sui-loro-siti-di-rimuoverlo

Test plan

  • Verify the app loads correctly in Chrome, Firefox, Safari
  • Verify no console errors related to missing polyfills
  • Check that features using replaceAll, IntersectionObserver, map, reduce still work as expected

The polyfill.io domain was compromised in 2024 and used to inject
malicious JavaScript via CDN responses. All loaded features (replaceAll,
IntersectionObserver, map, reduce) are natively supported by all modern
browsers — no alternative needed.
@currents-bot
Copy link

currents-bot bot commented Mar 9, 2026

Currents Tests: Tryber.me tests failed after 3m 46.8s

🟢 241 · 🔴 1 · ⚪️ 0 · 🟣 1

View Run Details

Run Details

  • Project: Tryber.me

  • Groups: 1

  • Framework: Playwright

  • Run Status: Failed

  • Commit: 3d594c9

  • Spec files: 13

  • Overall tests: 242

  • Duration: 3m 46.8s

  • Parallelization: 0

Failed Spec Files

Spec File Failures
e2e/wallet/table_net_gross.spec.ts 1


This message was posted automatically by currents.dev | Integration Settings

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant