Infrastructure/logto
0
Fork 0
mirror of https://github.com/logto-io/logto.git synced 2024-12-16 20:26:19 -05:00
Code Issues Activity

chore(core): update generation of SAML service provider entity id (#4943)

Browse source
This commit is contained in:
Darcy Ye 2023-11-23 10:56:41 +08:00 committed by GitHub
parent 5fd390a8ce
commit df6d0894ba
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 8 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
packages/core/src/sso/SamlConnector/index.ts
View file

@ -57,12 +57,14 @@ class SamlConnector {
tenantId: string,
ssoConnectorId: string
) {
const tenantEndpoint = getTenantEndpoint(tenantId, EnvSet.values);
this.assertionConsumerServiceUrl = buildAssertionConsumerServiceUrl(
getTenantEndpoint(tenantId, EnvSet.values),
tenantEndpoint,
ssoConnectorId
);
this.spEntityId = buildSpEntityId(EnvSet.values, tenantId, ssoConnectorId);
this.spEntityId = buildSpEntityId(tenantEndpoint, ssoConnectorId);
this.serviceProviderMetadata = {
entityId: this.spEntityId,

19
packages/core/src/sso/SamlConnector/utils.ts
View file

@ -1,6 +1,5 @@
import * as validator from '@authenio/samlify-node-xmllint';
import { ConnectorError, ConnectorErrorCodes } from '@logto/connector-kit';
import { type GlobalValues } from '@logto/shared';
import { type Optional, conditional, appendPath } from '@silverhand/essentials';
import { got } from 'got';
import * as saml from 'samlify';
@ -181,27 +180,17 @@ export const attributeMappingPostProcessor = (
};
/**
* Generate the entity id for the current SAML SSO connector using admin console path, current tenant id and connector id.
* Generate the entity id for the current SAML SSO connector using tenant endpoint path and connector id.
* Used URL-like entity id here since some identity providers will check the format of the entity id.
* See {@link https://spaces.at.internet2.edu/display/federation/saml-metadata-entityid} to know more details about how should `entityId` look like.
*
* @param globalValues Global setups
* @param tenantId Current tenant id.
* @param baseUrl Base endpoint for the current service
* @param connectorId Current connector id.
*
* @returns Entity id for the current SAML SSO connector.
*/
export const buildSpEntityId = (
globalValues: GlobalValues,
tenantId: string,
connectorId: string
) => {
const { isCloud, cloudUrlSet, adminUrlSet } = globalValues;
return appendPath(
isCloud ? cloudUrlSet.endpoint : adminUrlSet.endpoint,
tenantId,
`/enterprise-sso/${connectorId}`
).toString();
export const buildSpEntityId = (baseUrl: URL, connectorId: string) => {
return appendPath(baseUrl, `/enterprise-sso/${connectorId}`).toString();
};
/**