Flagsmith/flagsmith

API crash because of bad URL parameters

Open

#7.361 geöffnet am 28. Apr. 2026

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Python (3.475 Stars) (264 Forks)batch import
good first issueimprovement

Beschreibung

Acceptance criteria

  • URL matching or view logic prevents an error with a proper 4xx response.

Sentry Issue: FLAGSMITH-API-5NF

ValueError: invalid literal for int() with base 10: 'org_3COWhASRXfhcdxVrd0wEjpQp4dg'
  File "django/db/models/fields/__init__.py", line 2128, in get_prep_value
    return int(value)

ValueError: Field 'id' expected a number but got 'org_3COWhASRXfhcdxVrd0wEjpQp4dg'.
(17 additional frame(s) were not displayed)
...
  File "features/views.py", line 861, in get_queryset
    return super().get_queryset().filter(identity__pk=self.kwargs["identity_pk"])  # type: ignore[no-untyped-call]

Contributes to #6809

Contributor Guide

Tech Stack
pythondjangorest api
Domain
backend
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
half day
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
fresh
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
mostly clear
Voraussetzungen
Django viewsURL routing
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60
Research-Richtung
The error occurs in features/views.py line 861 in the get queryset method, where self.kwargs['identity pk'] is passed to filter. This value is expected to be an integer but receives a string (e.g., 'org ...'). Investigate the URL configuration to see how identity pk is captured; it may be matching incorrectly. Ensure that the URL pattern only accepts integer values for identity pk, or add validation in the view to return a 400 response for non integer inputs. Check the URL patterns in urls.py related to this view.