Skip to content

fix: review follow-up for upstream #806 - #6

Draft
dinakarjs wants to merge 1 commit into
review/source-806-20261025from
fix/review-806-20261025
Draft

dinakarjs wants to merge 1 commit into
review/source-806-20261025from
fix/review-806-20261025

Conversation

@dinakarjs

Copy link
Copy Markdown
Owner

A present gateway.agent_identity with null, non-object, missing, empty or non-string agent_id currently bypasses optional identity binding. This companion rejects those malformed present values while preserving the absent optional field and valid fixture.

Based exactly on agentrust-io#806 head 944c03c. Please adopt this signed commit into that PR; this draft is an adoption companion, not a competing upstream change.

Validation: unit suite 2547 passed/6 skipped; conformance+integration+confinement 114 passed/23 skipped. Ruff on src/tests, strict mypy and Bandit passed. All five malformed-identity parameter cases fail by assertion when just this fix is removed; valid/absent controls pass with the repair.

Signed-off-by: Srinivasa Consultants <78673153+dinakarjs@users.noreply.github.com>
@github-actions

github-actions Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

🟡 Contributor Check: MEDIUM

Check Result
Profile MEDIUM
Credential LOW
Overall MEDIUM

Automated check by AgenTrust Contributor Check.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-review:MEDIUM Contributor check flagged MEDIUM risk

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant