diff --git a/packages/backend/src/graphql/queries/list-saml-auth-providers.ee.js b/packages/backend/src/graphql/queries/list-saml-auth-providers.ee.js deleted file mode 100644 index 1e2ee21e..00000000 --- a/packages/backend/src/graphql/queries/list-saml-auth-providers.ee.js +++ /dev/null @@ -1,9 +0,0 @@ -import SamlAuthProvider from '../../models/saml-auth-provider.ee.js'; - -const listSamlAuthProviders = async () => { - const providers = await SamlAuthProvider.query().where({ active: true }); - - return providers; -}; - -export default listSamlAuthProviders; diff --git a/packages/backend/src/graphql/query-resolvers.js b/packages/backend/src/graphql/query-resolvers.js index ff26502d..97897970 100644 --- a/packages/backend/src/graphql/query-resolvers.js +++ b/packages/backend/src/graphql/query-resolvers.js @@ -13,7 +13,6 @@ import getPermissionCatalog from './queries/get-permission-catalog.ee.js'; import getSamlAuthProviderRoleMappings from './queries/get-saml-auth-provider-role-mappings.ee.js'; import getStepWithTestExecutions from './queries/get-step-with-test-executions.js'; import getUsers from './queries/get-users.js'; -import listSamlAuthProviders from './queries/list-saml-auth-providers.ee.js'; import testConnection from './queries/test-connection.js'; const queryResolvers = { @@ -32,7 +31,6 @@ const queryResolvers = { getSamlAuthProviderRoleMappings, getStepWithTestExecutions, getUsers, - listSamlAuthProviders, testConnection, }; diff --git a/packages/backend/src/graphql/schema.graphql b/packages/backend/src/graphql/schema.graphql index eb584aca..68d474b0 100644 --- a/packages/backend/src/graphql/schema.graphql +++ b/packages/backend/src/graphql/schema.graphql @@ -29,7 +29,6 @@ type Query { getNotifications: [Notification] getSamlAuthProviderRoleMappings(id: String!): [SamlAuthProvidersRoleMapping] getUsers(limit: Int!, offset: Int!): UserConnection - listSamlAuthProviders: [ListSamlAuthProvider] } type Mutation { @@ -637,13 +636,6 @@ type Usage { task: Int } -type ListSamlAuthProvider { - id: String - name: String - issuer: String - loginUrl: String -} - type Permission { id: String action: String diff --git a/packages/backend/src/helpers/authentication.js b/packages/backend/src/helpers/authentication.js index 448d8777..1e34a8fd 100644 --- a/packages/backend/src/helpers/authentication.js +++ b/packages/backend/src/helpers/authentication.js @@ -44,7 +44,6 @@ export const authenticationRules = { '*': isAuthenticatedRule, getConfig: allow, getNotifications: allow, - listSamlAuthProviders: allow, }, Mutation: { '*': isAuthenticatedRule, diff --git a/packages/web/src/components/SsoProviders/index.ee.jsx b/packages/web/src/components/SsoProviders/index.ee.jsx index 5689eda6..a1cb6557 100644 --- a/packages/web/src/components/SsoProviders/index.ee.jsx +++ b/packages/web/src/components/SsoProviders/index.ee.jsx @@ -3,19 +3,25 @@ import Paper from '@mui/material/Paper'; import Button from '@mui/material/Button'; import Stack from '@mui/material/Stack'; import Divider from '@mui/material/Divider'; + import useSamlAuthProviders from 'hooks/useSamlAuthProviders.ee'; import useFormatMessage from 'hooks/useFormatMessage'; + function SsoProviders() { const formatMessage = useFormatMessage(); - const { providers, loading } = useSamlAuthProviders(); - if (!loading && providers.length === 0) return null; + const { data, isLoading: isSamlAuthProvidersLoading } = + useSamlAuthProviders(); + const providers = data?.data; + + if (!isSamlAuthProvidersLoading && providers?.length === 0) return null; + return ( <> {formatMessage('loginPage.divider')} - {providers.map((provider) => ( + {providers?.map((provider) => (