Skip to content

Replace getOrganization with getCreOrganizationInfo in auth validation#377

Merged
timothyF95 merged 3 commits into
mainfrom
add-derived-wf-owner-to-rtc
Apr 17, 2026
Merged

Replace getOrganization with getCreOrganizationInfo in auth validation#377
timothyF95 merged 3 commits into
mainfrom
add-derived-wf-owner-to-rtc

Conversation

@timothyF95
Copy link
Copy Markdown
Contributor

Summary

  • Replaces the CLI auth validator's getOrganization query with getCreOrganizationInfo, which returns the org ID and the server-derived workflow owner address in a single call
  • Adds OrgID and DerivedWorkflowOwner fields to the runtime context, populated during auth validation
  • Removes client-side workflow owner derivation (GenerateWorkflowOwnerAddress) from the private registry deploy path — uses the server-provided value instead
  • Removes JWT claim parsing for org ID (GetOrgID()) in the deploy command and secrets browser flow — uses the org ID already set by the validator
  • Returns a clear error message when the account is not yet fully provisioned

@timothyF95 timothyF95 marked this pull request as ready for review April 16, 2026 11:13
@timothyF95 timothyF95 requested a review from a team as a code owner April 16, 2026 11:13
j-nowak
j-nowak previously approved these changes Apr 16, 2026
@timothyF95 timothyF95 added this pull request to the merge queue Apr 17, 2026
Merged via the queue into main with commit c5db971 Apr 17, 2026
22 checks passed
@timothyF95 timothyF95 deleted the add-derived-wf-owner-to-rtc branch April 17, 2026 02:38
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.

3 participants