Handle non-mapping request data in update_traits

[m-husnain-dev]

Description

This change prevents a crash when request.data is not a mapping before unpacking it with **request.data in update_traits().

Instead of raising a TypeError, the endpoint now returns a proper ValidationError for invalid request payloads.

Fixes #7951

[vercel]

@m-husnain-dev is attempting to deploy a commit to the Flagsmith Team on Vercel.

A member of the Team first needs to authorize it.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: a2eec89b-3dcd-48ad-ae0d-c5aeb4b98fb2

📥 Commits

Reviewing files that changed from the base of the PR and between da194ca and 9932350.

📒 Files selected for processing (1)

• api/edge_api/identities/views.py

---

📝 Walkthrough

Walkthrough

This change adds a validation check to the update_traits method in EdgeIdentityViewSet. Before processing trait data, the code now verifies that request.data is a dictionary (JSON object). If it is not, a ValidationError is raised with a descriptive message, preventing further processing of malformed payloads.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Changes

File	Change Summary
api/edge_api/identities/views.py	Added a type check in update_traits to raise ValidationError when request.data is not a dictionary.

Sequence Diagram(s)

sequenceDiagram
    participant Client
    participant EdgeIdentityViewSet
    participant TraitModel

    Client->>EdgeIdentityViewSet: PUT/POST trait data
    EdgeIdentityViewSet->>EdgeIdentityViewSet: check isinstance(request.data, dict)
    alt is dict
        EdgeIdentityViewSet->>TraitModel: construct trait instances
        TraitModel-->>EdgeIdentityViewSet: validated traits
        EdgeIdentityViewSet-->>Client: success response
    else not dict
        EdgeIdentityViewSet-->>Client: ValidationError
    end

Loading

Related issues: None specified.

Related PRs: None specified.

Suggested labels: api, bug-fix

Suggested reviewers: None specified.

---

🐰 A tiny check before the trait,
Guards the gate, keeps data straight,
If not a dict, it won't proceed,
A validation error's what you'll need,
Hops of safety, small but great!

---

_{Comment @coderabbitai help to get the list of available commands.}