Redirect direct visits to coverage-compass-policy-engine.vercel.app to policyengine.org

Abierto
#11 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
55/100
Tipo de issue
Nueva funcionalidad
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
next.js, typescript

Línea de trabajo

Start with frontend/next.config.ts, then inspect frontend/src/app/layout.tsx, frontend/public/sitemap.xml and frontend/public/robots.txt for the current URLs. Check for absolute calls to the Vercel host and follow the linked redirect guidance and tests. Done means direct-host requests redirect to the canonical policyengine.org URLs while proxied requests still work; the issue also calls for a production curl check and an audit update in policyengine-app-v2.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

policyengine.org serves this app at https://www.policyengine.org/us/coverage-compass: policyengine-app-v2's website/src/data/appZoneRoutes.ts proxies that path to https://coverage-compass-policy-engine.vercel.app. The host also serves every page itself, outside the site, so links to it keep turning up in posts, emails and READMEs. A visit straight to coverage-compass-policy-engine.vercel.app should land on the same page on policyengine.org.

The change

In frontend/next.config.ts, add three redirects. as const keeps the condition's type the literal "header": without it TypeScript widens it to string, which Next 16's NextConfig type rejects (TS2322, checked on Next 16.2.6).

const PUBLIC_URL = "https://www.policyengine.org/us/coverage-compass";
const directVisit = [
  { type: "header" as const, key: "x-forwarded-host", value: "coverage-compass-policy-engine\\.vercel\\.app" },
];

// in nextConfig:
async redirects() {
  return [
    // The host's root, outside the basePath.
    { source: "/", basePath: false, has: directVisit, destination: PUBLIC_URL, permanent: true },
    // The bare basePath. Without this rule Vercel sends it to PUBLIC_URL + "/", a second hop.
    { source: "/", has: directVisit, destination: PUBLIC_URL, permanent: true },
    { source: "/:path*", has: directVisit, destination: `${PUBLIC_URL}/:path*`, permanent: true },
  ];
},

Vercel sets x-forwarded-host to the host the visitor asked for, and overwrites any value the visitor sends. That's coverage-compass-policy-engine.vercel.app on a direct visit and www.policyengine.org on a request policyengine.org proxies. So these rules leave policyengine.org's own requests alone, and preview deployments too, since a preview has a host of its own. The pattern, how it was tested and its pitfalls: policyengine-app-v2 docs, "The direct-host redirect". PolicyEngine/ballot-measures-2026#8 adopts it with tests.

Canonical link and og:url

Once the host redirects, a page whose <link rel="canonical"> or og:url names coverage-compass-policy-engine.vercel.app names a URL that redirects back to it, and policyengine-app-v2's audit fails a redirecting zone whose page does. Today this page's canonical link and og:url name https://coverage-compass.policyengine.org, a subdomain that no longer resolves (checked 2026-10-29): frontend/src/app/layout.tsx sets it as metadataBase and in the JSON-LD, and frontend/public/sitemap.xml and frontend/public/robots.txt use it too. In the same change, point the canonical link and og:url at https://policyengine.org/us/coverage-compass, and the JSON-LD, sitemap and robots.txt at URLs under it.

Assets and API calls

These rules also redirect direct requests for the host's own assets and API routes. Pages served through policyengine.org load theirs through policyengine.org, so readers there never meet the redirect. But a browser blocks a cross-origin call whose redirect carries no CORS headers, so check that no code calls this host by absolute URL. A code search of the default branch for coverage-compass-policy-engine.vercel.app found nothing (2026-10-29).

Check it

/us/coverage-compass already serves this app on policyengine.org, so this can ship whenever it suits you. After the production deploy:

curl -sI "https://coverage-compass-policy-engine.vercel.app/us/coverage-compass"

That should answer 308 with location: https://www.policyengine.org/us/coverage-compass, and curl -sI "https://www.policyengine.org/us/coverage-compass" should still answer 200.

Then, in policyengine-app-v2, run node .github/scripts/audit-zone-direct-hosts.mjs, and add "/us/coverage-compass" to DIRECT_HOST_REDIRECTS there so the daily audit keeps the redirect in place.

🤖 Generated with Claude Code

Lenguaje dominante
TypeScript
Estrellas
1
Forks
0
Merge medio
2 h 1 min
PR fusionados (30 d)
1

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.