Skip to content

Fijar Supabase JS y añadir SRI a los recursos CDN - #145

Merged
yosef7 merged 1 commit into
mainfrom
fix/137-cdn-sri
Sep 30, 2026
Merged

yosef7 merged 1 commit into
mainfrom
fix/137-cdn-sri

Conversation

@yosef7

@yosef7 yosef7 commented Sep 27, 2026 •

Copy link
Copy Markdown
Member

Fija Supabase JS en 2.117.2 con una ruta UMD explícita y SHA-384 en registro, ponentes y voluntariado. Es la versión que resolvía el alias @2 al preparar el cambio; sus bytes coinciden con el paquete publicado en npm. Añade SHA-384 y crossorigin="anonymous" a Font Awesome 6.4.0 en las doce páginas de 2026.

El README documenta cómo actualizar versiones, regenerar hashes y verificar CORS. Una prueba de regresión detecta versiones flotantes, SRI/CORS ausentes y discrepancias entre páginas; CI ejecuta todos los tests.

Refs #137

Validación:

  • 23 tests aprobados; validador local y git diff --check correctos.
  • Hashes calculados sobre los archivos descargados y CORS permitido por ambos CDN.
  • SDK UMD real probado con respuestas simuladas para las tres rutas (asistentes, ponentes y voluntariado), sin escrituras remotas.
  • Validador externo: dos 404 preexistentes en URLs de preconexión de Google Fonts de 2026/about.html.

Preview verificado: https://deploy-preview-145--pycon.netlify.app

  • Chrome ejecuta supabase.createClient y aplica la hoja de Font Awesome con SRI en registro, ponentes y voluntariado.
  • El botón de ponencias permanece deshabilitado.
  • El preview apunta al mismo proyecto Supabase que el sitio público; no se realizaron escrituras remotas.
  • Netlify aprobó despliegue, cabeceras y redirecciones.
  • GitHub Actions falla por los mismos dos 404 preexistentes de Google Fonts, confirmados en el log del run 36327122622.

Pendiente del issue: envíos con persistencia en una base aislada configurada para el preview. Se solicitó al mantenedor identificar ese entorno; no se afirma persistencia en Supabase a partir de las pruebas simuladas. El issue se mantiene abierto.


Revisión conjunta de las PR pendientes: PR borrador #153 (NO FUSIONAR) muestra este cambio junto con los demás y documenta los conflictos resueltos y las pruebas. Esta PR individual conserva su aprobación y fusión propias.

@yosef7
yosef7 requested a review from a team September 27, 2026 14:46
@netlify

netlify Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for pycon ready!

Name Link
🔨 Latest commit e409d16
🔍 Latest deploy log https://app.netlify.com/projects/pycon/deploys/6ab92c3cc97ade0008cadf71
😎 Deploy Preview https://deploy-preview-145--pycon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@NoelSanz25 NoelSanz25 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verificado para realizar prueba de CDN

@yosef7
yosef7 merged commit 97186c9 into main Sep 30, 2026
4 of 5 checks passed
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.

2 participants