Auth/ PM-41503 & PM-41533 - Add openOrgInvite param to SDK registration finish call - #2953
Conversation
The SDK's `UserMasterPasswordRegistrationRequest` gains a new `openOrgInvite` field (`Option<RegistrationFinishOpenOrgInviteData>`) to support finishing registration via an open organization invite link. iOS does not use open-org-invite (that flow is web-only), so this passes `nil` at the sole call site to satisfy the new required constructor parameter.
🤖 Bitwarden Claude Code ReviewOverall Assessment: APPROVE This PR adds a single Code Review DetailsNo findings. Merge-order note (not a code finding): the Existing tests in |
…st model to password registration finish (#1363) ## 🎟️ Tracking <!-- Paste the link to the Jira or GitHub issue or otherwise describe / point to where this change is coming from. --> https://bitwarden.atlassian.net/browse/PM-41503 https://bitwarden.atlassian.net/browse/PM-41533 Server PR: bitwarden/server#8159 Server SDK Bindings Update PR: #1372 PRs that will merge shortly after that depend on this change: bitwarden/clients#22333 bitwarden/ios#2953 bitwarden/android#7255 ## 📔 Objective <!-- Describe what the purpose of this PR is, for example what bug you're fixing or new feature you're adding. --> Adds an optional `open_org_invite` field to the normal master-password open registration finish flow so the register-finish payload can carry the open-organization-invite data. Introduces a purpose-built `RegistrationFinishOpenOrgInviteData` request type and a new `RegistrationError::InvalidInput` variant for UUID parse failures.
🎟️ Tracking
https://bitwarden.atlassian.net/browse/PM-41503
https://bitwarden.atlassian.net/browse/PM-41533
Server PR: bitwarden/server#8159
Server SDK Bindings Update PR: bitwarden/sdk-internal#1372
SDK PR which wires up breaking changes: bitwarden/sdk-internal#1363
📔 Objective
The SDK's
UserMasterPasswordRegistrationRequestwill gain a newopenOrgInvitefield (Option<RegistrationFinishOpenOrgInviteData>) to support finishing registration via an open organization invite link once bitwarden/sdk-internal#1363 merges. iOS does not use open-org-invite (that flow is web-only), so this passesnilat the sole call site to satisfy the new required constructor parameter.📸 Screenshots
n/a