@sap/cds 5.8.4 → 5.9.2

package/libx/_runtime/hana/execute.js

@@ -4,12 +4,7 @@ const LOG = cds.log('hana|db|sql')
 const { HANA_TYPE_CONVERSION_MAP } = require('./conversion')
 const CustomBuilder = require('./customBuilder')
 const { sqlFactory } = require('../db/sql-builder/')
-const {
-  getPostProcessMapper,
-  getPropertyMapper,
-  getStructMapper,
-  postProcess
-} = require('../db/data-conversion/post-processing')
+const { getPostProcessMapper, postProcess } = require('../db/data-conversion/post-processing')
 const { createJoinCQNFromExpanded, hasExpand, rawToExpanded, expandV2 } = require('../db/expand')
 const {
   hasStreamInsert,
@@ -35,19 +30,6 @@ function _cqnToSQL(model, query, user, locale, txTimestamp) {
 
 const SANITIZE_VALUES = process.env.NODE_ENV === 'production' && cds.env.log.sanitize_values !== false
 
-function _getOutputParameters(stmt) {
-  const result = {}
-  const info = stmt.getParameterInfo()
-  for (let i = 0; i < info.length; i++) {
-    const param = info[i]
-    if (param.direction === 2) {
-      result[param.name] = stmt.getParameterValue(i)
-    }
-  }
-
-  return Object.keys(result).length > 0 ? result : undefined
-}
-
 const BINARY_TYPES = {
   12: 'BINARY',
   13: 'VARBINARY',
@@ -148,47 +130,20 @@ function _executeAsPreparedStatement(dbc, sql, values, reject, resolve) {
       }
     }
 
-    if (cds.env.features.new_call_prodecure) {
-      // procedure call metadata
-      let outParameters
-      const isProcedureCall = _isProcedureCall(sql)
-      if (isProcedureCall) {
-        try {
-          const procedureName = _getProcedureName(sql)
-          outParameters = await _getProcedureMetadata(procedureName, dbc)
-        } catch (e) {
-          LOG._warn && LOG.warn('Unable to fetch procedure metadata due to error:', e)
-        }
+    // procedure call metadata
+    let outParameters
+    const isProcedureCall = _isProcedureCall(sql)
+    if (isProcedureCall) {
+      try {
+        const procedureName = _getProcedureName(sql)
+        outParameters = await _getProcedureMetadata(procedureName, dbc)
+      } catch (e) {
+        LOG._warn && LOG.warn('Unable to fetch procedure metadata due to error:', e)
       }
-
-      // on @sap/hana-client, we need to use execQuery in case of calling procedures
-      stmt[isProcedureCall && dbc.name !== 'hdb' ? 'execQuery' : 'exec'](values, function (err, rows, ...args) {
-        if (err) {
-          stmt.drop(() => {})
-          err.query = sql
-          if (values) err.values = SANITIZE_VALUES ? ['***'] : values
-          return reject(err)
-        }
-
-        let result
-        if (isProcedureCall) {
-          result =
-            dbc.name === 'hdb'
-              ? _hdbGetResultForProcedure(rows, args, outParameters)
-              : _hcGetResultForProcedure(stmt, rows, outParameters)
-        } else {
-          result = rows
-        }
-
-        stmt.drop(() => {})
-
-        resolve(result)
-      })
-
-      return
     }
 
-    stmt.exec(values, function (err, rows, procedureReturn) {
+    // on @sap/hana-client, we need to use execQuery in case of calling procedures
+    stmt[isProcedureCall && dbc.name !== 'hdb' ? 'execQuery' : 'exec'](values, function (err, rows, ...args) {
       if (err) {
         stmt.drop(() => {})
         err.query = sql
@@ -196,13 +151,19 @@ function _executeAsPreparedStatement(dbc, sql, values, reject, resolve) {
         return reject(err)
       }
 
-      let result = rows
-      if (dbc.name !== 'hdb') {
-        result = _getOutputParameters(stmt) || rows
+      let result
+      if (isProcedureCall) {
+        result =
+          dbc.name === 'hdb'
+            ? _hdbGetResultForProcedure(rows, args, outParameters)
+            : _hcGetResultForProcedure(stmt, rows, outParameters)
+      } else {
+        result = rows
       }
 
       stmt.drop(() => {})
-      resolve(procedureReturn || result)
+
+      resolve(result)
     })
   })
 }
@@ -236,13 +197,13 @@ function _executeSimpleSQL(dbc, sql, values) {
   })
 }
 
-function _executeSelectSQL(dbc, sql, values, isOne, postMapper, propertyMapper, objStructMapper) {
+function _executeSelectSQL(dbc, sql, values, isOne, postMapper) {
   return _executeSimpleSQL(dbc, sql, values).then(result => {
     if (isOne) {
       result = result.length > 0 ? result[0] : null
     }
 
-    return postProcess(result, postMapper, propertyMapper, objStructMapper)
+    return postProcess(result, postMapper)
   })
 }
 
@@ -273,17 +234,9 @@ function executeSelectCQN(model, dbc, query, user, locale, txTimestamp) {
   }
 
   const { sql, values = [] } = _cqnToSQL(model, query, user, locale, txTimestamp)
-  const propertyMapper = getPropertyMapper(model, query, true)
-
-  return _executeSelectSQL(
-    dbc,
-    sql,
-    values,
-    query.SELECT.one,
-    getPostProcessMapper(HANA_TYPE_CONVERSION_MAP, model, query),
-    propertyMapper,
-    getStructMapper(model, query, propertyMapper)
-  )
+  const postProcessMapper = getPostProcessMapper(HANA_TYPE_CONVERSION_MAP, model, query)
+
+  return _executeSelectSQL(dbc, sql, values, query.SELECT.one, postProcessMapper)
 }
 
 function _getValuesProxy(values) {
@@ -327,11 +280,28 @@ function executeInsertCQN(model, dbc, query, user, locale, txTimestamp) {
   }
 
   return _executeSimpleSQL(dbc, sql, values).then(affectedRows => {
+    const entriesOrRows = query.INSERT.entries || query.INSERT.rows
+    const affectedRowsCount = Array.isArray(affectedRows)
+      ? affectedRows.reduce((sum, rows) => sum + rows, 0)
+      : affectedRows
+    if (entriesOrRows && entriesOrRows.length !== affectedRowsCount) {
+      LOG._warn &&
+        LOG.warn(
+          `INSERT input deviates from affected rows (input: ${entriesOrRows.length}, affectedRows: ${affectedRowsCount})`,
+          {
+            sql,
+            args: values && values.length,
+            values,
+            query
+          }
+        )
+      throw new Error('Possible data loss by INSERT into HANA db. Please, update a corresponding HANA driver.')
+    }
     // InsertResult needs an object per row with its values
     // query.INSERT.values -> one row
     if (query.INSERT.values) return [{ affectedRows: 1, values: [values] }]
     // query.INSERT.entries or .rows -> multiple rows
-    if (query.INSERT.entries || query.INSERT.rows) return values.map(v => ({ affectedRows: 1, values: v }))
+    if (entriesOrRows) return values.map(v => ({ affectedRows: 1, values: v }))
     // INSERT into SELECT
     return [{ affectedRows }]
   })

package/libx/_runtime/hana/pool.js

@@ -97,7 +97,7 @@ function factory4(creds, tenant) {
 /*
  * default generic-pool config
  */
-const defaultConfig = { min: 0, max: 100, testOnBorrow: true }
+const defaultConfig = { min: 0, max: 100, testOnBorrow: true, fifo: false }
 
 const _getPoolConfig = function () {
   const { pool: poolConfig } = cds.env.requires.db

package/libx/_runtime/hana/streaming.js

@@ -18,7 +18,8 @@ const streamExtension = _loadStreamExtensionIfNeeded()
 
 function hasStreamInsert(insert, model) {
   if (!model) return true
-  const into = model.definitions[ensureNoDraftsSuffix(insert.into)]
+  const name = insert.into.ref ? insert.into.ref[0] : insert.into
+  const into = model.definitions[ensureNoDraftsSuffix(name)]
   if (!into) return false
 
   if (insert.entries && insert.entries.length > 0) {

package/libx/_runtime/index.js

@@ -5,12 +5,12 @@ module.exports = {
       return this._odatav4 || (this._odatav4 = require('./cds-services/adapter/odata-v4/to'))
     },
 
-    get rest() {
-      if (!this._rest) {
-        if (global.cds.env.features.rest_new_adapter) this._rest = require('../rest')
-        else this._rest = require('./cds-services/adapter/rest/to')
-      }
-      return this._rest
+    get old_rest() {
+      return this._old_rest || (this._old_rest = require('./cds-services/adapter/rest/to'))
+    },
+
+    get new_rest() {
+      return this._new_rest || (this._new_rest = require('../rest'))
     }
   },
 

package/libx/_runtime/messaging/AMQPWebhookMessaging.js

@@ -1,8 +1,8 @@
 const cds = require('../cds')
-const LOG = cds.log('messaging')
 const MessagingService = require('./service.js')
 const { queueName } = require('./common-utils/naming-conventions')
 const optionsApp = require('../common/utils/vcap.js')
+const normalizeIncomingMessage = require('./common-utils/normalizeIncomingMessage')
 
 class AMQPWebhookMessaging extends MessagingService {
   async init() {
@@ -33,25 +33,9 @@ class AMQPWebhookMessaging extends MessagingService {
       const management = this.getManagement()
       if (!opt.doNotDeploy) this.queued(management.createQueueAndSubscriptions.bind(management))()
       this.queued(this.listenToClient.bind(this))(async (_topic, _payload, _other, { done, failed }) => {
-        const event = _topic
-        // Some messaging systems don't adhere to the standard that the payload has a `data` property.
-        // For these cases, we interpret the whole payload as `data`.
-        let data, headers
-        if (typeof _payload === 'object' && 'data' in _payload) {
-          data = _payload.data
-          headers = { ..._payload }
-          delete headers.data
-        } else {
-          data = _payload
-          headers = {}
-        }
-        const msg = {
-          event,
-          data,
-          headers,
-          inbound: true,
-          ...(_other || {})
-        }
+        const msg = Object.assign(normalizeIncomingMessage(_payload), _other || {})
+        msg.event = _topic
+
         if (!msg._) msg._ = {}
         msg._.topic = _topic
         try {
@@ -61,7 +45,7 @@ class AMQPWebhookMessaging extends MessagingService {
           // In case of AMQP and Solace, the `failed` callback must be called
           // with an error, otherwise there are problems with the redelivery count.
           failed(new Error('processing failed'))
-          LOG.error('ERROR occured in asynchronous event processing:', e)
+          this.LOG.error('ERROR occured in asynchronous event processing:', e)
         }
       })
     }

package/libx/_runtime/messaging/Outbox.js

@@ -19,7 +19,7 @@ class OutboxService extends cds.Service {
     this.emit = async function (...args) {
       const msg = typeof args[0] === 'object' ? args[0] : { event: args[0], data: args[1], headers: args[2] }
       const context = this.context || cds.context
-      if (this.options.outbox && context && typeof context.on === 'function') {
+      if (this.options.outbox && context) {
         const outboxOpts = Object.assign(
           {},
           (typeof cds.requires.outbox === 'object' && cds.requires.outbox) || {},
@@ -38,7 +38,7 @@ class OutboxService extends cds.Service {
           try {
             await this._emitImmediate(msg)
           } catch (e) {
-            LOG._error && LOG.error('Emit failed', { event: msg.event, cause: e })
+            LOG.error('Emit failed', { event: msg.event, cause: e })
             // opts.crashOnError is not official!!!
             if (isUnrecoverable(this, e) && outboxOpts.crashOnError !== false) process.exit(1)
           }

package/libx/_runtime/messaging/common-utils/AMQPClient.js

@@ -1,17 +1,8 @@
 const cds = require('../../cds.js')
-const LOG = cds.log('messaging')
 const ClientAmqp = require('@sap/xb-msg-amqp-v100').Client
 const { connect, disconnect } = require('./connections')
 const { hasPersistentOutbox } = require('../outbox/utils')
 
-const _JSONorString = string => {
-  try {
-    return JSON.parse(string)
-  } catch (e) {
-    return string
-  }
-}
-
 const addDataListener = (client, queue, prefix, cb) =>
   new Promise((resolve, reject) => {
     const source = `${prefix}${queue}`
@@ -19,8 +10,7 @@ const addDataListener = (client, queue, prefix, cb) =>
       .receiver(queue)
       .attach(source)
       .on('data', async raw => {
-        const buffer = raw.payload && Buffer.concat(raw.payload.chunks)
-        const payload = buffer && _JSONorString(buffer.toString())
+        const payload = raw.payload && Buffer.concat(raw.payload.chunks).toString()
         const topic =
           raw.source &&
           raw.source.properties &&
@@ -36,7 +26,7 @@ const addDataListener = (client, queue, prefix, cb) =>
 
 const sender = (client, optionsApp) => client.sender(`${optionsApp.appName}-${optionsApp.appID}`)
 
-const emit = ({ data, event: topic, headers = {} }, stream, prefix) =>
+const emit = ({ data, event: topic, headers = {} }, stream, prefix, LOG) =>
   new Promise((resolve, reject) => {
     LOG._info && LOG.info('Emit', { topic })
     const message = { ...headers, data }
@@ -69,7 +59,7 @@ class AMQPClient {
     this.client = new ClientAmqp(this.optionsAMQP)
     this.sender = sender(this.client, this.service.optionsApp)
     this.stream = this.sender.attach('')
-    return connect(this.client, this.keepAlive)
+    return connect(this.client, this.service.LOG, this.keepAlive)
   }
 
   async disconnect() {
@@ -84,7 +74,7 @@ class AMQPClient {
     // REVISIT: Is this a robust way to find out if the connection is working?
     if (hasPersistentOutbox(this.service, cds.context && cds.context.tenant) && !this.sender.opened())
       throw new Error('AMQP: Sender is not open')
-    await emit(msg, this.stream, this.prefix.topic)
+    await emit(msg, this.stream, this.prefix.topic, this.service.LOG)
     if (!this.keepAlive) return this.disconnect()
   }
 

package/libx/_runtime/messaging/common-utils/connections.js

@@ -1,11 +1,9 @@
-const cds = require('../../cds')
-const LOG = cds.log('messaging')
 const waitingTime = require('./waitingTime')
 
-const _connectUntilConnected = (client, x) => {
+const _connectUntilConnected = (client, LOG, x) => {
   const _waitingTime = waitingTime(x)
   setTimeout(() => {
-    connect(client, true)
+    connect(client, LOG, true)
       .then(() => {
         LOG._warn && LOG.warn('Reconnected to Enterprise Messaging Client')
       })
@@ -14,12 +12,12 @@ const _connectUntilConnected = (client, x) => {
           LOG.warn(
             `Connection to Enterprise Messaging Client lost: Reconnecting in ${Math.round(_waitingTime / 1000)} s`
           )
-        _connectUntilConnected(client, x + 1)
+        _connectUntilConnected(client, LOG, x + 1)
       })
   }, _waitingTime)
 }
 
-const connect = (client, keepAlive) => {
+const connect = (client, LOG, keepAlive) => {
   return new Promise((resolve, reject) => {
     client
       .once('connected', function () {
@@ -37,7 +35,7 @@ const connect = (client, keepAlive) => {
           client.once('disconnected', () => {
             client.removeAllListeners('error')
             client.removeAllListeners('connected')
-            _connectUntilConnected(client, 0)
+            _connectUntilConnected(client, LOG, 0)
           })
         }
 

package/libx/_runtime/messaging/common-utils/normalizeIncomingMessage.js

@@ -0,0 +1,30 @@
+const _JSONorString = string => {
+  try {
+    return JSON.parse(string)
+  } catch (e) {
+    return string
+  }
+}
+
+// Some messaging systems don't adhere to the standard that the payload has a `data` property.
+// For these cases, we interpret the whole payload as `data`.
+const normalizeIncomingMessage = message => {
+  const _payload = typeof message === 'object' ? message : _JSONorString(message)
+  let data, headers
+  if (typeof _payload === 'object' && 'data' in _payload) {
+    data = _payload.data
+    headers = { ..._payload }
+    delete headers.data
+  } else {
+    data = _payload
+    headers = {}
+  }
+
+  return {
+    data,
+    headers,
+    inbound: true
+  }
+}
+
+module.exports = normalizeIncomingMessage

package/libx/_runtime/messaging/enterprise-messaging-shared.js

@@ -34,7 +34,8 @@ class EnterpriseMessagingShared extends AMQPWebhookMessaging {
       queueName,
       subscribedTopics: this.subscribedTopics,
       alternativeTopics: this.alternativeTopics,
-      namespace: this.options.credentials && this.options.credentials.namespace
+      namespace: this.options.credentials && this.options.credentials.namespace,
+      LOG: this.LOG
     })
     return this.management
   }

package/libx/_runtime/messaging/enterprise-messaging-utils/EMManagement.js

@@ -1,6 +1,4 @@
 const authorizedRequest = require('../common-utils/authorizedRequest')
-const cds = require('../../cds.js')
-const LOG = cds.log('messaging')
 const sleep = require('util').promisify(setTimeout)
 
 const _getWebhookName = queueName => queueName
@@ -19,7 +17,8 @@ class EMManagement {
     subscribedTopics,
     maxRetries,
     subdomain,
-    namespace
+    namespace,
+    LOG
   }) {
     this.subdomain = subdomain
     this.options = optionsManagement
@@ -33,6 +32,7 @@ class EMManagement {
     this.maxRetries = maxRetries === undefined ? 10 : maxRetries
     this.subdomainInfo = this.subdomain ? `(subdomain: ${this.subdomain})` : ''
     this.namespace = namespace
+    this.LOG = LOG
   }
 
   async getQueue(queueName = this.queueName) {
@@ -42,8 +42,11 @@ class EMManagement {
       path: `/hub/rest/api/v1/management/messaging/queues/${encodeURIComponent(queueName)}`,
       oa2: this.options.oa2,
       attemptInfo: () =>
-        LOG._info &&
-        LOG.info('Get queue', this.subdomain ? { queue: queueName, subdomain: this.subdomain } : { queue: queueName }),
+        this.LOG._info &&
+        this.LOG.info(
+          'Get queue',
+          this.subdomain ? { queue: queueName, subdomain: this.subdomain } : { queue: queueName }
+        ),
       errMsg: `Queue "${queueName}" could not be retrieved ${this.subdomainInfo}`,
       target: { kind: 'QUEUE', queue: queueName },
       tokenStore: this
@@ -57,7 +60,8 @@ class EMManagement {
       uri: this.options.uri,
       path: `/hub/rest/api/v1/management/messaging/queues`,
       oa2: this.options.oa2,
-      attemptInfo: () => LOG._info && LOG.info('Get queues', this.subdomain ? { subdomain: this.subdomain } : {}),
+      attemptInfo: () =>
+        this.LOG._info && this.LOG.info('Get queues', this.subdomain ? { subdomain: this.subdomain } : {}),
       errMsg: `Queues could not be retrieved ${this.subdomainInfo}`,
       target: { kind: 'QUEUE' },
       tokenStore: this
@@ -73,8 +77,8 @@ class EMManagement {
       oa2: this.options.oa2,
       dataObj: this.queueConfig,
       attemptInfo: () =>
-        LOG._info &&
-        LOG.info(
+        this.LOG._info &&
+        this.LOG.info(
           'Create queue',
           this.subdomain ? { queue: queueName, subdomain: this.subdomain } : { queue: queueName }
         ),
@@ -91,8 +95,8 @@ class EMManagement {
       path: `/hub/rest/api/v1/management/messaging/queues/${encodeURIComponent(queueName)}`,
       oa2: this.options.oa2,
       attemptInfo: () =>
-        LOG._info &&
-        LOG.info(
+        this.LOG._info &&
+        this.LOG.info(
           'Delete queue',
           this.subdomain ? { queue: queueName, subdomain: this.subdomain } : { queue: queueName }
         ),
@@ -109,8 +113,8 @@ class EMManagement {
       path: `/hub/rest/api/v1/management/messaging/queues/${encodeURIComponent(queueName)}/subscriptions`,
       oa2: this.options.oa2,
       attemptInfo: () =>
-        LOG._info &&
-        LOG.info(
+        this.LOG._info &&
+        this.LOG.info(
           'Get subscriptions',
           this.subdomain ? { queue: queueName, subdomain: this.subdomain } : { queue: queueName }
         ),
@@ -130,8 +134,8 @@ class EMManagement {
       )}/subscriptions/${encodeURIComponent(topicPattern)}`,
       oa2: this.options.oa2,
       attemptInfo: () =>
-        LOG._info &&
-        LOG.info(
+        this.LOG._info &&
+        this.LOG.info(
           'Create subscription',
           this.subdomain
             ? { topic: topicPattern, queue: queueName, subdomain: this.subdomain }
@@ -152,8 +156,8 @@ class EMManagement {
       )}/subscriptions/${encodeURIComponent(topicPattern)}`,
       oa2: this.options.oa2,
       attemptInfo: () =>
-        LOG._info &&
-        LOG.info(
+        this.LOG._info &&
+        this.LOG.info(
           'Delete subscription',
           this.subdomain
             ? { topic: topicPattern, queue: queueName, subdomain: this.subdomain }
@@ -173,8 +177,8 @@ class EMManagement {
       path: `/messagingrest/v1/subscriptions/${encodeURIComponent(webhookName)}`,
       oa2: this.optionsMessagingREST.oa2,
       attemptInfo: () =>
-        LOG._info &&
-        LOG.info(
+        this.LOG._info &&
+        this.LOG.info(
           'Get webhook',
           this.subdomain
             ? { webhook: webhookName, queue: queueName, subdomain: this.subdomain }
@@ -195,8 +199,8 @@ class EMManagement {
       path: `/messagingrest/v1/subscriptions/${encodeURIComponent(webhookName)}`,
       oa2: this.optionsMessagingREST.oa2,
       attemptInfo: () =>
-        LOG._info &&
-        LOG.info(
+        this.LOG._info &&
+        this.LOG.info(
           'Delete webhook',
           this.subdomain
             ? { webhook: webhookName, queue: queueName, subdomain: this.subdomain }
@@ -245,8 +249,8 @@ class EMManagement {
       oa2: this.optionsMessagingREST.oa2,
       dataObj,
       attemptInfo: () =>
-        LOG._info &&
-        LOG.info(
+        this.LOG._info &&
+        this.LOG.info(
           'Create webhook',
           this.subdomain
             ? { webhook: webhookName, queue: queueName, subdomain: this.subdomain }
@@ -266,8 +270,8 @@ class EMManagement {
       path: `/messagingrest/v1/subscriptions/${encodeURIComponent(webhookName)}`,
       oa2: this.optionsMessagingREST.oa2,
       attemptInfo: () =>
-        LOG._info &&
-        LOG.info(
+        this.LOG._info &&
+        this.LOG.info(
           'Delete webhook',
           this.subdomain
             ? { webhook: webhookName, queue: queueName, subdomain: this.subdomain }
@@ -280,7 +284,7 @@ class EMManagement {
   }
 
   async createQueueAndSubscriptions() {
-    LOG._info && LOG.info(`Create messaging artifacts ${this.subdomainInfo}`)
+    this.LOG._info && this.LOG.info(`Create messaging artifacts ${this.subdomainInfo}`)
 
     const created = await this.createQueue()
     if (created && created.statusCode === 200) {
@@ -297,7 +301,7 @@ class EMManagement {
         for (const [s, _] of this.subscribedTopics) {
           if (existingSubscriptions.some(e => s === e)) unchangedSubs.push(s)
         }
-        LOG._info && LOG.info('Unchanged subscriptions', unchangedSubs, ' ', this.subdomainInfo)
+        this.LOG._info && this.LOG.info('Unchanged subscriptions', unchangedSubs, ' ', this.subdomainInfo)
         await Promise.all([
           ...obsoleteSubs.map(s => this.deleteSubscription(s)),
           ...additionalSubs.map(async s => this._createSubscription(s))
@@ -331,7 +335,7 @@ class EMManagement {
   }
 
   async undeploy() {
-    LOG._info && LOG.info(`Delete messaging artifacts ${this.subdomainInfo}`)
+    this.LOG._info && this.LOG.info(`Delete messaging artifacts ${this.subdomainInfo}`)
     await this.deleteQueue()
     if (this.optionsMessagingREST) await this.deleteWebhook()
   }
@@ -342,7 +346,7 @@ class EMManagement {
       uri: this.options.uri,
       path: `/hub/rest/api/v1/management/messaging/readinessCheck`,
       oa2: this.options.oa2,
-      attemptInfo: () => LOG._info && LOG.info(`Readiness Check ${this.subdomainInfo}`),
+      attemptInfo: () => this.LOG._info && this.LOG.info(`Readiness Check ${this.subdomainInfo}`),
       errMsg: `Readiness Check failed ${this.subdomainInfo}`,
       target: { kind: 'READINESSCHECK' },
       tokenStore: this
@@ -365,8 +369,10 @@ class EMManagement {
           }
           const retryAfter = e.response && e.response.headers && e.response.headers['retry-after']
           const _waitingPeriod = waitingPeriod || (retryAfter && Number(retryAfter) * 1000) || 120 * 1000
-          LOG._info &&
-            LOG.info(`Readiness Check failed ${this.subdomainInfo}, retrying in ${_waitingPeriod / 1000} seconds...`)
+          this.LOG._info &&
+            this.LOG.info(
+              `Readiness Check failed ${this.subdomainInfo}, retrying in ${_waitingPeriod / 1000} seconds...`
+            )
           await sleep(_waitingPeriod)
           await check()
         } else {