Skip to content

sdk: append the payer Permission account on feed and user commands - #4228

Open
martinsander00 wants to merge 1 commit into
mainfrom
ms/infra-2343
Open

sdk: append the payer Permission account on feed and user commands#4228
martinsander00 wants to merge 1 commit into
mainfrom
ms/infra-2343

Conversation

@martinsander00

Copy link
Copy Markdown
Contributor

Summary of Changes

  • The Rust SDK appends the payer Permission account on doublezero feed create, feed delete, feed update, user delete, and user update when that account exists and the serviceability program owns it.
  • A signer that only holds feed-authority or user-admin on that account was denied with program error 0x8 (NotAllowed) because those commands never sent the extra account.
  • doublezero user create is unchanged. That instruction counts accounts to detect an optional tenant.

Part of https://github.com/malbeclabs/infra/issues/2343

Testing Verification

  • Feed create has a missing-account case and a present-account case. The present case expects the Permission account last on the instruction.
  • Feed delete, feed update, user delete, and user update stub a missing Permission account so the instruction matches the builder with no extra account.

@martinsander00
martinsander00 requested a review from a team August 21, 2026 23:52
@martinsander00
martinsander00 force-pushed the ms/infra-2343 branch 4 times, most recently from 6863d60 to 7604a8b Compare August 22, 2026 00:18
A signer that only holds those grants on its Permission account was denied on feed create/delete/update and user delete/update because those sends never included the trailing account.
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