Repository navigation
fix: el 404 del rechazo devolvía la fila, y el hub vencía tareas de hoy - #88
Closed
JoseSerpaCode wants to merge 1 commit into
Closed
JoseSerpaCode wants to merge 1 commit into
JoseSerpaCode wants to merge 1 commit into
Conversation
Dos fallos pequeños de riesgo alto. - `friends/reject/[id].ts` devolvía `dbState` con el `SELECT *` de la amistad a quien acababa de fallar la comprobación de permisos, más un `console.log` de cada intento. Sus dos hermanos responden con la frase a secas. - El contador de vencidas del hub comparaba en UTC, así que en UTC-5 marcaba como vencida una entrega de hoy a partir de las siete de la tarde. `lib/fechas.ts` ya tenía `vencida()` y su comentario advierte del caso nombrando Bogotá; el hub reimplementó la comparación a mano. Las dos pruebas verificadas reintroduciendo su fallo. La de fechas fija la hora del sistema, que es lo único que reproduce el de UTC. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Owner
Author
|
Consolidada en una sola PR con las cinco correcciones y una versión coherente. Se abrió como |
3 tasks done
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bloque A del plan, los dos primeros: pequeños, de riesgo alto y sin decisiones de por medio.
A.3 — Fuga de datos al rechazar una solicitud
api/friends/reject/[id].tsdevolvía en el 404 undbStatecon elSELECT *entero de la amistad —action_user_id, el id de la otra persona, las marcas de tiempo — a quien acababa de fallar la comprobación de permisos. Había además unconsole.logde cada intento. Depuración que se quedó puesta.Sus dos hermanos,
accept/[id].tsycancel/[id].ts, responden'Not Found or Unauthorized'a secas. Este era el único de los tres fuera del patrón.A.2 — El hub marcaba vencida una tarea que vence hoy
Hub.astro:78comparaba connew Date().toISOString(), que da el día en UTC. En Bogotá, a partir de las siete de la tarde, el servidor ya cree que es mañana.Lo llamativo:
lib/fechas.tsdocumenta ese caso exacto en su comentario —nombrando Bogotá— y exponevencida()desde entonces.IssueCard.astroya la usa. Este contador reimplementó la comparación por su cuenta y se trajo el fallo de vuelta.Plan de pruebas
npm run lint,npm run typecheck— limpios🤖 Generated with Claude Code