From 11b7ceb1cb365f61449d1772c59a753eeb179c8e Mon Sep 17 00:00:00 2001 From: Konstantinos Feretos Date: Wed, 15 Jun 2022 12:40:40 +0300 Subject: [PATCH] feat(grpc-sdk,commons,security): cli security client platform, ping route security middleware exceptions (#207) --- .../grpc-sdk/src/constants/enums/PlatformTypesEnum.ts | 1 + packages/commons/src/constants/enums/PlatformTypesEnum.ts | 1 + packages/security/src/handlers/client-validation/index.ts | 8 ++++++-- 3 files changed, 8 insertions(+), 2 deletions(-) diff --git a/libraries/grpc-sdk/src/constants/enums/PlatformTypesEnum.ts b/libraries/grpc-sdk/src/constants/enums/PlatformTypesEnum.ts index 320f74a36..2bee378b6 100644 --- a/libraries/grpc-sdk/src/constants/enums/PlatformTypesEnum.ts +++ b/libraries/grpc-sdk/src/constants/enums/PlatformTypesEnum.ts @@ -6,4 +6,5 @@ export enum PlatformTypesEnum { WINDOWS = 'WINDOWS', MACOS = 'MACOS', LINUX = 'LINUX', + CLI = 'CLI', } diff --git a/packages/commons/src/constants/enums/PlatformTypesEnum.ts b/packages/commons/src/constants/enums/PlatformTypesEnum.ts index 320f74a36..2bee378b6 100644 --- a/packages/commons/src/constants/enums/PlatformTypesEnum.ts +++ b/packages/commons/src/constants/enums/PlatformTypesEnum.ts @@ -6,4 +6,5 @@ export enum PlatformTypesEnum { WINDOWS = 'WINDOWS', MACOS = 'MACOS', LINUX = 'LINUX', + CLI = 'CLI', } diff --git a/packages/security/src/handlers/client-validation/index.ts b/packages/security/src/handlers/client-validation/index.ts index 2e4b34a58..65eb4b43c 100644 --- a/packages/security/src/handlers/client-validation/index.ts +++ b/packages/security/src/handlers/client-validation/index.ts @@ -34,8 +34,12 @@ export class ClientValidator { if (isNil((req as ConduitModelOptions).conduit)) (req as ConduitModelOptions).conduit = {}; const { clientid, clientsecret } = req.headers; - // if incoming call is a webhook or an admin call - if (req.path.indexOf('/hook') === 0 || req.path.indexOf('/admin') === 0) { + // Exclude webhooks, admin calls and http pings + if ( + req.path.indexOf('/hook') === 0 || + req.path.indexOf('/admin') === 0 || + ['/', '/health'].includes(req.path) + ) { return next(); }