@commercetools/connect-payments-sdk 0.6.0 → 0.7.0

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @commercetools/connect-payments-sdk
 
+## 0.7.0
+
+### Minor Changes
+
+- e471faf: Adding logs in the status handler
+
 ## 0.6.0
 
 ### Minor Changes

package/dist/api/handlers/status.handler.d.ts

@@ -1,4 +1,5 @@
 import { CommercetoolsAuthorizationService } from '../../commercetools';
+import { Logger } from '../../logger';
 import { HandlerResponse } from './types/handler.type';
 type HealthCheckStatus = {
     status: 'OK' | 'Partially Available' | 'Unavailable';
@@ -11,13 +12,21 @@ export type HealthCheckResult = {
     name: string;
     status: 'UP' | 'DOWN';
     details?: object;
+    message?: string;
 };
 export type HealthCheck = () => Promise<HealthCheckResult> | HealthCheckResult;
-export declare const statusHandler: (options: {
+export type StatusHandlerOpts = {
     timeout: number;
     checks: HealthCheck[];
     metadataFn?: () => Promise<object> | object;
-}) => () => Promise<HandlerResponse<HealthCheckStatus>>;
+    log: Logger;
+};
+/**
+ * Handler to check the status of the service
+ * @param opts
+ * @returns
+ */
+export declare const statusHandler: (opts: StatusHandlerOpts) => () => Promise<HandlerResponse<HealthCheckStatus>>;
 /**
  * Check if CoCo permissions are available
  * @param opts

package/dist/api/handlers/status.handler.js

@@ -1,19 +1,38 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.healthCheckCommercetoolsPermissions = exports.statusHandler = void 0;
-const statusHandler = (options) => async () => {
+/**
+ * Handler to check the status of the service
+ * @param opts
+ * @returns
+ */
+const statusHandler = (opts) => async () => {
     const status = {
         timestamp: new Date().toISOString(),
         version: process.env.npm_package_version ?? '0.0.0',
     };
-    if (options.metadataFn) {
-        status.metadata = await options.metadataFn();
+    if (opts.metadataFn) {
+        status.metadata = await opts.metadataFn();
     }
     let timeoutId = null;
-    const timeoutPromise = new Promise((_, reject) => (timeoutId = setTimeout(() => reject(new Error('Timeout')), options.timeout)));
+    const timeoutPromise = new Promise((_, reject) => (timeoutId = setTimeout(() => reject(new Error('Timeout')), opts.timeout)));
     try {
-        const results = await Promise.race([Promise.all(options.checks.map((check) => check())), timeoutPromise]);
-        if (results.findIndex((result) => result.status === 'DOWN') !== -1) {
+        const results = await Promise.race([Promise.all(opts.checks.map((check) => check())), timeoutPromise]);
+        let allDown = true;
+        let atLeastOneDown = false;
+        results.forEach((result) => {
+            if (result.status === 'DOWN') {
+                atLeastOneDown = true;
+                opts.log.error({ details: result.details }, result.message ?? `Health check for ${result.name} failed, check details`);
+            }
+            if (result.status === 'UP') {
+                allDown = false;
+            }
+        });
+        if (allDown) {
+            status.status = 'Unavailable';
+        }
+        else if (atLeastOneDown) {
             status.status = 'Partially Available';
         }
         else {
@@ -21,11 +40,14 @@ const statusHandler = (options) => async () => {
         }
         status.checks = results;
         return {
-            status: 200,
+            status: allDown ? 503 : 200,
             body: status,
         };
     }
-    catch (error) {
+    catch (e) {
+        if (e instanceof Error && e.message === 'Timeout') {
+            opts.log.error({ err: e }, 'Health check timed out');
+        }
         status.status = 'Unavailable';
         status.checks = [];
         return {
@@ -60,6 +82,7 @@ const healthCheckCommercetoolsPermissions = (opts) => async () => {
     return {
         name: 'CoCo Permissions',
         status: 'DOWN',
+        message: `CoCo permissions are not correct, expected scopes: ${opts.requiredPermissions.join(' ')}, actual scopes: ${token.scope}`,
         details: {
             expectedScopes: opts.requiredPermissions,
             actualScopes: token.scope,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@commercetools/connect-payments-sdk",
-  "version": "0.6.0",
+  "version": "0.7.0",
   "description": "Payment SDK for commercetools payment connectors",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -15,9 +15,9 @@
   ],
   "license": "ISC",
   "dependencies": {
-    "@commercetools-backend/loggers": "22.23.3",
-    "@commercetools/platform-sdk": "7.7.0",
-    "@commercetools/sdk-client-v2": "2.4.1",
+    "@commercetools-backend/loggers": "22.27.0",
+    "@commercetools/platform-sdk": "7.8.0",
+    "@commercetools/sdk-client-v2": "2.5.0",
     "jsonwebtoken": "9.0.2",
     "jwks-rsa": "3.1.0",
     "lodash": "4.17.21",