PUT user-profile returns 500 when user_id is missing from the payload
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
问题出在 policyengine_api/routes/user_profile_routes.py 中的 update_user_profile，其中 payload.pop("user_id") 没有默认值，会抛出 KeyError。请阅读现有的 user_id 为 null 时返回 400 并附带 "Payload must include user_id" 的分支，让缺失键的情况遵循同样的处理。完成标准是：不含 user_id 的 payload 得到与 null 情况相同的 400 响应，并且测试覆盖两种情况。
由索引模型根据 Issue 内容生成。
描述
The PUT /<country_id>/user-profile route (update_user_profile in policyengine_api/routes/user_profile_routes.py) reads the ID with payload.pop("user_id"), with no default. A payload without a user_id key raises KeyError, which the generic error handler turns into a 500.
The route already returns 400 ("Payload must include user_id") when user_id is present but null, so a missing key should get the same 400.
Expected
A payload without user_id gives 400 with the same message as a null user_id, for example by using payload.pop("user_id", None). Add a test for both cases.
Notes
- Found while documenting the code for the ruff docstring rules; behaviour was left unchanged there.
- The route has a TODO pointing at #2054 (request validation), which is now closed.
- 主要语言
- Python
- 星标
- 18
- 派生
- 33
- 平均合并
- 1 天 15 小时
- 30 天内合并 PR
- 18
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
PolicyEngine/policyengine-api 的其他 Issue
-
难度 1/5 1 小时以内 新手友好度 82/100
PolicyEngine/policyengine-api#3910 ·
维护者通常 1 天内回复
-
Enable ruff security, logging and debugging rules (S, T20, LOG, G, DTZ, ERA, TD, FIX, EXE)可能已有人在做 @anth-volk 今天认领。 未关闭
难度 4/5 3-5 天 新手友好度 18/100
PolicyEngine/policyengine-api#3914 ·
维护者通常 1 天内回复
-
难度 4/5 3-5 天 新手友好度 38/100
PolicyEngine/policyengine-api#3912 ·
维护者通常 1 天内回复
-
validate_country prints "Validating country" on every request可能已有人在做 @anth-volk 今天认领。 未关闭
难度 1/5 1 小时以内 新手友好度 25/100
PolicyEngine/policyengine-api#3909 ·
维护者通常 1 天内回复
-
难度 2/5 半天 新手友好度 48/100
PolicyEngine/policyengine-api#3908 ·
维护者通常 1 天内回复
查看 PolicyEngine/policyengine-api 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 82/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 70/100
onyx-dot-app/onyx#16016 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 1 天内回复
-
black --check exits 0 and prints All done when a multi file run is interrupted可能已有人在做 @jiteshgulati 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 2 天内回复
-
Non-positive worker counts wait for results without starting workers可能已有人在做 @IlyaKhalafi 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 85/100
维护者通常 1 天内回复