Skip to content

fix(agentconnect-ui): an approval past its expiry is expired, not sent on to agentcard - #228

Merged
JamesLawton merged 1 commit into
stagingfrom
fix/expired-authorization
Oct 29, 2026
Merged

JamesLawton merged 1 commit into
stagingfrom
fix/expired-authorization

Conversation

@JamesLawton

Copy link
Copy Markdown
Collaborator

Kernel keeps reporting an AgentCard authorization as awaiting_approval after its expires_at. An old checkout's approval link therefore still went to AgentCard, whose page refused it with "Refusing to pay: malformed payment endpoint" (yesterday's Noordcode authorization, expired at 18:10 UTC, opened today). Agents also kept treating that checkout as pending and wouldn't start a new one.

  • An authorization past expires_at counts as expired: browser_checkout_status reports expired, the browser closes and the approval is marked failed (nothing charged).
  • Approving on /approve first settles the checkout, so an expired one shows "didn't go through, nothing was charged" instead of opening AgentCard.

Tests: 290 pass, one new.

🤖 Generated with Claude Code

…t on to agentcard

Kernel keeps reporting an AgentCard authorization as awaiting approval after it
expires, so an old checkout's link opened AgentCard's page on a payment it can't
take ("malformed payment endpoint"). Check expires_at; approving an expired one
now settles it as failed and frees the agent to start again.
@claude

claude Bot commented Oct 29, 2026

Copy link
Copy Markdown

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@JamesLawton
JamesLawton merged commit 00021af into staging Oct 29, 2026
1 check passed
@JamesLawton
JamesLawton deleted the fix/expired-authorization branch October 29, 2026 12:11
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