Skip to content

[GHSA-q63x-9pfm-mjx4] Feast before 0.63.0 contains an unsafe deserialization... - #9191

Open
PercevalFox wants to merge 1 commit into
PercevalFox/advisory-improvement-9191from
PercevalFox-GHSA-q63x-9pfm-mjx4
Open

[GHSA-q63x-9pfm-mjx4] Feast before 0.63.0 contains an unsafe deserialization...#9191
PercevalFox wants to merge 1 commit into
PercevalFox/advisory-improvement-9191from
PercevalFox-GHSA-q63x-9pfm-mjx4

Conversation

@PercevalFox

Copy link
Copy Markdown

Updates

  • Affected products
  • CVSS v3
  • CVSS v4
  • Summary

Comments
This advisory is unreviewed and has no package mapping, so it does not surface to users of the affected package.

The repository feast-dev/feast publishes the PyPI package feast.

The advisory describes the registry server deserializing the user_defined_function.body field of an OnDemandFeatureView via dill.loads() before any authorization check. The fix commit (835cda8e) adds a skip_udf parameter to the from_proto methods and passes skip_udf=True from sdk/python/feast/registry_server.py for FeatureView, OnDemandFeatureView and StreamFeatureView, so the registry server no longer deserializes caller-supplied UDFs. The commit message is generic ("Fix issue with apply feature view") but the diff is unambiguous on this point.

I verified the boundary against the published PyPI wheels:

  • 0.62.0 ships feast/registry_server.py calling from_proto without skip_udf
  • 0.63.0 ships the same file passing skip_udf=True

This matches the "before 0.63.0" wording in the description and the release tag already listed in the references (https://github.com/feast-dev/feast/releases/tag/v0.63.0).

Proposed range: introduced 0, fixed 0.63.0.

The CVSS v4 vector was rewritten to omit the trailing Not Defined (X) metrics, which the form's validator rejects. Semantically identical per the CVSS v4.0 specification, no change to the severity assessment. CVSS v3 left untouched.

The advisory also had no title. The proposed one follows the database's naming convention and reflects the existing CWE-502 classification.

@github-actions
github-actions Bot changed the base branch from main to PercevalFox/advisory-improvement-9191 August 23, 2026 01:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant