Flagsmith/flagsmith

500 when calling /api/v1/features/featurestates/{pk}/ with empty environment

Open

#6,597 创建于 2026年1月27日

在 GitHub 查看
 (0 评论) (0 反应) (0 负责人)Python (3,475 star) (264 fork)batch import
apigood first issue

描述

Sentry Issue: FLAGSMITH-API-5GQ

AttributeError: 'NoneType' object has no attribute 'id'
(8 additional frame(s) were not displayed)
...
  File "features/serializers.py", line 578, in validate_environment
    if self.instance and self.instance.environment_id != environment.id:  # type: ignore[union-attr]

Acceptance criteria

  • /api/v1/features/featurestates/{pk}/ responds with HTTP 400 if environment is null in the payload.

贡献者指南

技术栈
pythondjangorest api
领域
backendapi
议题类型
bug
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
under 1 hour
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
fresh
清晰度议题是否清楚说明期望改动、验收标准和下一步。
clear
前置要求
PythonDjango REST frameworkserializer validation
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
70
研究方向
The bug is in features/serializers.py at line 578 in the validate environment method. When environment is null, it tries to access environment.id, causing AttributeError. The fix should add a check for None before accessing id, and return a validation error (HTTP 400) as per acceptance criteria. No linked PRs or comments. The Sentry issue (FLAGSMITH API 5GQ) provides more context.