Skip to content

fix(push): trim de la VAPID public key (applicationServerKey inválido) - #103

Merged
wikichaves merged 1 commit into
mainfrom
claude/pwa-notifications-guest-role-4z7mdb
Jul 23, 2026
Merged

fix(push): trim de la VAPID public key (applicationServerKey inválido)#103
wikichaves merged 1 commit into
mainfrom
claude/pwa-notifications-guest-role-4z7mdb

Conversation

@wikichaves

Copy link
Copy Markdown
Owner

Qué se hizo

El toggle de push tiraba Failed to execute 'subscribe' on 'PushManager': The provided applicationServerKey is not valid en Android, con las 3 env vars VAPID ya seteadas en Vercel.

Causa probable: un espacio o salto de línea al final del valor de NEXT_PUBLIC_VAPID_PUBLIC_KEY (artefacto de pegado común en Vercel) → urlBase64ToUint8Array decodifica un largo incorrecto → el browser rechaza la key.

Fix: .trim() defensivo sobre la key antes de decodificar.

Validación

npm run lint ✓ · npx tsc --noEmit

🤖 Generated with Claude Code


Generated by Claude Code

Un espacio o salto de línea al final del valor de NEXT_PUBLIC_VAPID_PUBLIC_KEY
(artefacto de pegado común en Vercel) hacía que la clave decodificara a un
largo incorrecto y el browser rechazara la suscripción con
"applicationServerKey is not valid". Trim defensivo antes de decodificar.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HXHQeq3LpQiRp2zmvXghf2
@wikichaves
wikichaves enabled auto-merge (squash) July 23, 2026 02:31
@vercel

vercel Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
tero-bot Ready Ready Preview, Comment Jul 23, 2026 2:32am

Request Review

@wikichaves
wikichaves merged commit 42261ce into main Jul 23, 2026
3 checks passed
@wikichaves
wikichaves deleted the claude/pwa-notifications-guest-role-4z7mdb branch July 23, 2026 02:33
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