From 8056c8116112ba266e47ac429d59b8ea6bf9a718 Mon Sep 17 00:00:00 2001 From: peintnermax Date: Wed, 23 Aug 2023 08:24:12 +0200 Subject: [PATCH] idp build fix --- .../register/idp/[provider]/success/page.tsx | 19 +++++++----- apps/login/lib/zitadel.ts | 29 ++++++++++--------- packages/zitadel-server/src/index.ts | 8 ++--- 3 files changed, 30 insertions(+), 26 deletions(-) diff --git a/apps/login/app/(login)/register/idp/[provider]/success/page.tsx b/apps/login/app/(login)/register/idp/[provider]/success/page.tsx index 1dc4792b2f1..f7b55fcffe5 100644 --- a/apps/login/app/(login)/register/idp/[provider]/success/page.tsx +++ b/apps/login/app/(login)/register/idp/[provider]/success/page.tsx @@ -1,10 +1,10 @@ import { ProviderSlug } from "#/lib/demos"; -import { addHumanUser, server } from "#/lib/zitadel"; +import { server } from "#/lib/zitadel"; import Alert, { AlertType } from "#/ui/Alert"; import { AddHumanUserRequest, IDPInformation, - RetrieveIdentityProviderInformationResponse, + RetrieveIdentityProviderIntentResponse, user, IDPLink, } from "@zitadel/server"; @@ -27,8 +27,8 @@ const PROVIDER_MAPPING: { // organisation: Organisation | undefined; profile: { displayName: idp.rawInformation?.User?.name ?? "", - firstName: idp.rawInformation?.User?.given_name ?? "", - lastName: idp.rawInformation?.User?.family_name ?? "", + givenName: idp.rawInformation?.User?.given_name ?? "", + familyName: idp.rawInformation?.User?.family_name ?? "", }, idpLinks: [idpLink], }; @@ -49,8 +49,8 @@ const PROVIDER_MAPPING: { // organisation: Organisation | undefined; profile: { displayName: idp.rawInformation?.name ?? "", - firstName: idp.rawInformation?.name ?? "", - lastName: idp.rawInformation?.name ?? "", + givenName: idp.rawInformation?.name ?? "", + familyName: idp.rawInformation?.name ?? "", }, idpLinks: [idpLink], }; @@ -64,8 +64,11 @@ function retrieveIDP( ): Promise { const userService = user.getUser(server); return userService - .retrieveIdentityProviderInformation({ intentId: id, token: token }, {}) - .then((resp: RetrieveIdentityProviderInformationResponse) => { + .retrieveIdentityProviderIntent( + { idpIntentId: id, idpIntentToken: token }, + {} + ) + .then((resp: RetrieveIdentityProviderIntentResponse) => { return resp.idpInformation; }); } diff --git a/apps/login/lib/zitadel.ts b/apps/login/lib/zitadel.ts index 0f7559d56ef..66beb96b22b 100644 --- a/apps/login/lib/zitadel.ts +++ b/apps/login/lib/zitadel.ts @@ -26,15 +26,16 @@ import { LoginSettings, GetLoginSettingsResponse, ListAuthenticationMethodTypesResponse, - StartIdentityProviderFlowRequest, - StartIdentityProviderFlowResponse, - RetrieveIdentityProviderInformationRequest, - RetrieveIdentityProviderInformationResponse, + StartIdentityProviderIntentRequest, + StartIdentityProviderIntentResponse, + RetrieveIdentityProviderIntentRequest, + RetrieveIdentityProviderIntentResponse, GetAuthRequestResponse, GetAuthRequestRequest, CreateCallbackRequest, CreateCallbackResponse, RequestChallenges, + AddHumanUserRequest, } from "@zitadel/server"; export const zitadelConfig: ZitadelServerOptions = { @@ -190,10 +191,10 @@ export async function addHumanUser( ): Promise { const userService = user.getUser(server); - const payload = { + const payload: Partial = { email: { email }, username: email, - profile: { firstName, lastName }, + profile: { givenName: firstName, familyName: lastName }, }; return userService .addHumanUser( @@ -212,11 +213,11 @@ export async function addHumanUser( export async function startIdentityProviderFlow( server: ZitadelServer, - { idpId, urls }: StartIdentityProviderFlowRequest -): Promise { + { idpId, urls }: StartIdentityProviderIntentRequest +): Promise { const userService = user.getUser(server); - return userService.startIdentityProviderFlow({ + return userService.startIdentityProviderIntent({ idpId, urls, }); @@ -224,13 +225,13 @@ export async function startIdentityProviderFlow( export async function retrieveIdentityProviderInformation( server: ZitadelServer, - { intentId, token }: RetrieveIdentityProviderInformationRequest -): Promise { + { idpIntentId, idpIntentToken }: RetrieveIdentityProviderIntentRequest +): Promise { const userService = user.getUser(server); - return userService.retrieveIdentityProviderInformation({ - intentId, - token, + return userService.retrieveIdentityProviderIntent({ + idpIntentId, + idpIntentToken, }); } diff --git a/packages/zitadel-server/src/index.ts b/packages/zitadel-server/src/index.ts index a2d7e8a9b43..1d802fbd4fc 100644 --- a/packages/zitadel-server/src/index.ts +++ b/packages/zitadel-server/src/index.ts @@ -71,10 +71,10 @@ export { ListAuthenticationMethodTypesResponse, ListAuthenticationMethodTypesRequest, AuthenticationMethodType, - StartIdentityProviderFlowRequest, - StartIdentityProviderFlowResponse, - RetrieveIdentityProviderInformationRequest, - RetrieveIdentityProviderInformationResponse, + StartIdentityProviderIntentRequest, + StartIdentityProviderIntentResponse, + RetrieveIdentityProviderIntentRequest, + RetrieveIdentityProviderIntentResponse, } from "./proto/server/zitadel/user/v2alpha/user_service"; export { SetHumanPasswordResponse,