CodeForPhilly / CodeForPhilly/codeforphilly-ng

idempotency: wire Idempotency-Key plugin into mutating endpoints

未关闭
#149 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
TypeScript
星标
1
派生
1
平均合并
5 天 3 小时
30 天内合并 PR
9

描述

The idempotency plugin (`apps/api/src/plugins/idempotency.ts`) is implemented and registered, but **no route handler calls it** — mutating endpoints don't honor the `Idempotency-Key` header yet.

`specs/api/conventions.md#idempotency` documents the target contract and now carries a status note marking it not-yet-wired (surfaced by the spec-drift audit).

**Do:** wire idempotency into the at-risk mutating endpoints — start with `POST /api/projects/:slug/updates` (the double-tap case the spec cites) — plus per-route tests. Then remove the status caveat from conventions.md.

贡献指南

这个仓库没有索引到贡献指南

调研方向

Read apps/api/src/plugins/idempotency.ts and the idempotency section of specs/api/conventions.md first, then locate the handler for POST /api/projects/:slug/updates. Run or add the per-route tests for the double-tap case, wire the plugin into the at-risk mutating endpoints, and remove the not-yet-wired status caveat once the contract is covered.

由索引模型根据 Issue 内容生成。

评估

技术栈
typescript
领域
api, backend
Issue 类型
功能
难度
4/5
预计耗时
3-5 天
活跃度
冷清
描述清晰度
基本清楚
新手友好度
68/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。