diff --git a/backend/src/ee/services/pam-resource/redis/redis-resource-factory.ts b/backend/src/ee/services/pam-resource/redis/redis-resource-factory.ts index 45908b783f..0838c65a7d 100644 --- a/backend/src/ee/services/pam-resource/redis/redis-resource-factory.ts +++ b/backend/src/ee/services/pam-resource/redis/redis-resource-factory.ts @@ -46,8 +46,6 @@ const makeRedisConnection = ( ): RedisResourceConnection => { const { connectionDetails, username, password } = config; const { sslEnabled, sslRejectUnauthorized, sslCertificate } = connectionDetails; - const actualUsername = username ?? connectionDetails.username ?? TEST_CONNECTION_USERNAME; - const actualPassword = password ?? connectionDetails.password ?? TEST_CONNECTION_PASSWORD; let client: Redis | null = null; @@ -55,8 +53,6 @@ const makeRedisConnection = ( return new Redis({ host: "localhost", port: proxyPort, - username: actualUsername, - password: actualPassword, connectTimeout: EXTERNAL_REQUEST_TIMEOUT, commandTimeout: EXTERNAL_REQUEST_TIMEOUT, ...(sslEnabled && { @@ -75,14 +71,6 @@ const makeRedisConnection = ( validate: async (connectOnly) => { try { client = createClient(); - - // Test authentication - const result = await client.auth(actualUsername, actualPassword); - if (result !== "OK") { - throw new BadRequestError({ message: `Redis authentication failed: ${result as string}` }); - } - - // Test connection with a simple command await client.ping(); } catch (error) { if (connectOnly) { diff --git a/backend/src/ee/services/pam-resource/redis/redis-resource-schemas.ts b/backend/src/ee/services/pam-resource/redis/redis-resource-schemas.ts index 6ad4cd6eaa..fce01b5fd1 100644 --- a/backend/src/ee/services/pam-resource/redis/redis-resource-schemas.ts +++ b/backend/src/ee/services/pam-resource/redis/redis-resource-schemas.ts @@ -15,8 +15,6 @@ import { export const RedisResourceConnectionDetailsSchema = z.object({ host: z.string().trim().min(1).max(255), port: z.coerce.number(), - username: z.string().trim().min(1).max(63).optional(), - password: z.string().trim().max(256).optional(), sslEnabled: z.boolean(), sslRejectUnauthorized: z.boolean(), sslCertificate: z diff --git a/frontend/src/pages/pam/PamResourcesPage/components/PamResourceForm/RedisResourceForm.tsx b/frontend/src/pages/pam/PamResourcesPage/components/PamResourceForm/RedisResourceForm.tsx index d35496a41a..6c9ed6400b 100644 --- a/frontend/src/pages/pam/PamResourcesPage/components/PamResourceForm/RedisResourceForm.tsx +++ b/frontend/src/pages/pam/PamResourcesPage/components/PamResourceForm/RedisResourceForm.tsx @@ -6,8 +6,8 @@ import { z } from "zod"; import { Button, ModalClose } from "@app/components/v2"; import { PamResourceType, TRedisResource } from "@app/hooks/api/pam"; -import { GenericResourceFields, genericResourceFieldsSchema } from "./GenericResourceFields"; import { RedisResourceFields } from "./shared/RedisResourceFields"; +import { GenericResourceFields, genericResourceFieldsSchema } from "./GenericResourceFields"; type Props = { resource?: TRedisResource; @@ -19,8 +19,6 @@ const formSchema = genericResourceFieldsSchema.extend({ connectionDetails: z.object({ host: z.string().trim().min(1, "Host required"), port: z.coerce.number().default(6379), - username: z.string().trim().optional(), - password: z.string().trim().optional(), sslEnabled: z.boolean().default(true), sslRejectUnauthorized: z.boolean().default(true), sslCertificate: z @@ -44,8 +42,6 @@ export const RedisResourceForm = ({ resource, onSubmit }: Props) => { connectionDetails: { host: "", port: 6379, - username: undefined, - password: undefined, sslEnabled: true, sslRejectUnauthorized: true, sslCertificate: undefined diff --git a/frontend/src/pages/pam/PamResourcesPage/components/PamResourceForm/shared/RedisResourceFields.tsx b/frontend/src/pages/pam/PamResourcesPage/components/PamResourceForm/shared/RedisResourceFields.tsx index dce79cb5a6..9b331133d0 100644 --- a/frontend/src/pages/pam/PamResourcesPage/components/PamResourceForm/shared/RedisResourceFields.tsx +++ b/frontend/src/pages/pam/PamResourcesPage/components/PamResourceForm/shared/RedisResourceFields.tsx @@ -69,38 +69,6 @@ export const RedisResourceFields = ({ setSelectedTabIndex, selectedTabIndex }: P )} /> -