Skip to content

mpa: add custom_payload and PayloadProvider hook - #623

Open
sfc-gh-ikryvanos wants to merge 3 commits into
mainfrom
mpa-custom-payload
Open

mpa: add custom_payload and PayloadProvider hook#623
sfc-gh-ikryvanos wants to merge 3 commits into
mainfrom
mpa-custom-payload

Conversation

@sfc-gh-ikryvanos

Copy link
Copy Markdown
Collaborator

Extend Action with an optional google.protobuf.Any custom_payload included in the action hash, and add a registerable PayloadProvider callback that can populate Store requests before they are persisted.

Extend Action with an optional google.protobuf.Any custom_payload included
in the action hash, and add a registerable PayloadProvider callback that
can populate Store requests before they are persisted.

Co-authored-by: Cursor <cursoragent@cursor.com>

@snowflake-security-bot snowflake-security-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Snowflake Security Review

Security grade: A — Passed

No security findings after adjudication. This PR passes the Snowflake Security Review.

📊 6 of 9 files (3 tests skipped) · 2,196 lines reviewed · 6 candidates → 1 kept · retrieval: on

Clarifies that the context flag bypasses MPA client interceptors only,
not server-side approval enforcement.

@snowflake-security-bot snowflake-security-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Snowflake Security Review

Security grade: A — Passed

No security findings after adjudication. This PR passes the Snowflake Security Review.

📊 6 of 9 files (3 tests skipped) · 2,196 lines reviewed · 7 candidates → 3 kept · retrieval: on

custom_payload is set by the requesting client, so bound its serialized
size (16 KiB) to keep a single Store call from bloating the in-memory
action store.

@snowflake-security-bot snowflake-security-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Snowflake Security Review

Security grade: A — Passed

No security findings after adjudication. This PR passes the Snowflake Security Review.

📊 6 of 9 files (3 tests skipped) · 2,208 lines reviewed · 11 candidates → 1 kept · 1 duplicate collapsed · retrieval: on

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