fix(web): same-origin check on cookie-authenticated kit and collection writes
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 72/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
Línea de trabajo
Read apps/web/app/lib/auth/same-origin.ts, then trace the write paths in kit-http.ts and collection-http.ts. Check how those routes distinguish session-cookie authentication from API-key clients, and run the route tests. Done means each route group rejects a foreign Origin with 403 for cookie-authenticated writes while same-origin and header-less requests pass and API-key clients keep working.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Follow-up to #101, which covered favorite/vote/rate/review. The personal kit (kit-http.ts) and collection (collection-http.ts) write paths accept session cookies without crossOriginRefusal (apps/web/app/lib/auth/same-origin.ts). They are currently shielded by the mandatory Content-Type: application/json (and preflight for PATCH/DELETE), so this is defence in depth, not an open hole. Apply the check only when the request is authenticated by the session cookie, so API-key clients keep working; add a test per route group: a foreign Origin returns 403, while same-origin and header-less requests pass. Found during review of #112.
- Lenguaje dominante
- TypeScript
- Estrellas
- 189
- Forks
- 40
- Merge medio
- 7 h 31 min
- PR fusionados (30 d)
- 43
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de nextlevelbuilder/skillx
-
ai-handle
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
nextlevelbuilder/skillx#105 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
nextlevelbuilder/skillx#28 ·
Los mantenedores suelen responder en 1 día
-
refactor
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
nextlevelbuilder/skillx#27 ·
Los mantenedores suelen responder en 1 día
-
feat(quality-packs): QGP foundation packs, private overrides và three-way rebase giữ nguyên opt-outAbiertoenhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 18/100
nextlevelbuilder/skillx#108 ·
Los mantenedores suelen responder en 1 día
-
feat(private-library): publish skills/kits riêng tư và đồng bộ đa thiết bị, kèm DoD với AgentKitAbiertoenhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 10/100
nextlevelbuilder/skillx#107 ·
Los mantenedores suelen responder en 1 día
Todos los issues de nextlevelbuilder/skillx
Issues similares
-
bug P3 UI/UX
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
StellarState/SS-backend#712 ·
Los mantenedores suelen responder en 1 día
-
area: backend bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
snapotter-hq/SnapOtter#3473 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
gitroomhq/postiz-app#2304 ·
Los mantenedores suelen responder en 1 día
-
Resolve CI maintenance findingsAbiertoharlan-agent-ready-to-implement routine:ci-review
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
harlan-zw/unlighthouse#454 · 2 comentarios ·
Los mantenedores suelen responder en 1 día