Skip to content

Commit 85b540c

Browse files
committed
fix AuthProviderService.createAuthProvider
1 parent c201696 commit 85b540c

File tree

1 file changed

+5
-9
lines changed

1 file changed

+5
-9
lines changed

components/server/src/api/auth-provider-service-api.ts

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -42,14 +42,13 @@ export class AuthProviderServiceAPI implements ServiceImpl<typeof AuthProviderSe
4242
request: CreateAuthProviderRequest,
4343
_: HandlerContext,
4444
): Promise<CreateAuthProviderResponse> {
45-
const ownerId = request.owner.case === "ownerId" ? request.owner.value : "";
4645
const organizationId = request.owner.case === "organizationId" ? request.owner.value : "";
4746

48-
if (!uuidValidate(organizationId) && !uuidValidate(ownerId)) {
49-
throw new ConnectError("organizationId or ownerId is required", Code.InvalidArgument);
50-
}
51-
5247
if (organizationId) {
48+
if (!uuidValidate(organizationId)) {
49+
throw new ConnectError("organizationId or ownerId is required", Code.InvalidArgument);
50+
}
51+
5352
const result = await this.authProviderService.createOrgAuthProvider(ctxUserId(), {
5453
organizationId,
5554
host: request.host,
@@ -87,10 +86,7 @@ export class AuthProviderServiceAPI implements ServiceImpl<typeof AuthProviderSe
8786
});
8887
}
8988

90-
async listAuthProviders(
91-
request: ListAuthProvidersRequest,
92-
_: HandlerContext,
93-
): Promise<ListAuthProvidersResponse> {
89+
async listAuthProviders(request: ListAuthProvidersRequest, _: HandlerContext): Promise<ListAuthProvidersResponse> {
9490
const target = request.id;
9591
const ownerId = target.case === "userId" ? target.value : "";
9692
const organizationId = target.case === "organizationId" ? target.value : "";

0 commit comments

Comments
 (0)