ac-support-connector 1.0.12 → 1.0.14

package/.github/workflows/node.js.yml

@@ -16,21 +16,15 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [20.x, 22.x]
+        node-version: [22.x, 24.x]
         # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
-        redis-version: [6]
 
     steps:
-    - uses: actions/checkout@v3
+    - uses: actions/checkout@v5
     - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v3
+      uses: actions/setup-node@v5
       with:
         node-version: ${{ matrix.node-version }}
-
-    - name: Start Redis
-      uses: supercharge/redis-github-action@1.4.0
-      with:
-        redis-version: ${{ matrix.redis-version }}    
     
     - run: yarn install
     - run: yarn run test

package/CHANGELOG.md

@@ -1,3 +1,39 @@
+## [1.0.14](https://github.com/admiralcloud/ac-support-connector/compare/v1.0.13..v1.0.14) (2026-04-09 11:27:07)
+
+
+### Bug Fix
+
+
+* **Connector:** Improvements after CoPilot review | MP | [e09a77aa4871c89a05383f0f03b076a6ef2f63c3](https://github.com/admiralcloud/ac-support-connector/commit/e09a77aa4871c89a05383f0f03b076a6ef2f63c3)    
+Improvements after CoPilot review  
+Related issues:
+* **Connector:** Add 2 new parameters - throwError, logger | MP | [982c057541f6e3cef22a661c0eb51403a899b2ef](https://github.com/admiralcloud/ac-support-connector/commit/982c057541f6e3cef22a661c0eb51403a899b2ef)    
+Set throwError as true per default, so errors are propagated. Allow custom loggers, default to console (as before)  
+Related issues:
+## [1.0.13](https://github.com/admiralcloud/ac-support-connector/compare/v1.0.12..v1.0.13) (2026-04-04 10:19:59)
+
+
+### Bug Fix
+
+
+* **Connector:** Package updaets | MP | [1bd12738508d496c03b217445bc7ce33e3e1b3bd](https://github.com/admiralcloud/ac-support-connector/commit/1bd12738508d496c03b217445bc7ce33e3e1b3bd)    
+Package updates  
+Related issues:
+* **Connector:** Package updates | MP | [a616f404236d18ab8d1afa9cc21f8d07bbcc1fb6](https://github.com/admiralcloud/ac-support-connector/commit/a616f404236d18ab8d1afa9cc21f8d07bbcc1fb6)    
+Package updates  
+Related issues:
+### Tests
+
+
+* **Connector:** Remove real Redis from test | MP | [f50d09c6e5f2959216dfb7454f2733e0fb2dfc4e](https://github.com/admiralcloud/ac-support-connector/commit/f50d09c6e5f2959216dfb7454f2733e0fb2dfc4e)    
+Mock Redis in test  
+Related issues:
+### Chores
+
+
+* **Connector:** Updated Github actions | MP | [b779c602f162af94547c968e267bafb611b6af8f](https://github.com/admiralcloud/ac-support-connector/commit/b779c602f162af94547c968e267bafb611b6af8f)    
+Updated Github actions  
+Related issues:
 
 ## [1.0.12](https://github.com/admiralcloud/ac-support-connector/compare/v1.0.11..v1.0.12) (2026-03-21 11:21:38)
 

package/README.md

@@ -1,7 +1,7 @@
 # AC Support Connector
 This module sends unified support payloads to SQS. A dedicated service (not part of the connector) then processes those SQS messages and delivers the support messages to different channels (e.g. Teams, a helpdesk, etc) 
 
-[![Node.js CI](https://github.com/AdmiralCloud/ac-geoip/actions/workflows/node.js.yml/badge.svg)](https://github.com/AdmiralCloud/ac-support-connector/actions/workflows/node.js.yml)
+[![Node.js CI](https://github.com/AdmiralCloud/ac-geoip/actions/workflows/node.js.yml/badge.svg)](https://github.com/AdmiralCloud/ac-support-connector/actions/workflows/node.js.yml) [![CodeQL](https://github.com/AdmiralCloud/ac-support-connector/actions/workflows/github-code-scanning/codeql/badge.svg)](https://github.com/AdmiralCloud/ac-support-connector/actions/workflows/github-code-scanning/codeql)
 
 # Breaking changes 
 ## Version 1

package/index.js

@@ -9,7 +9,7 @@ module.exports = {
 
   aws: {},
 
-  init: async function({ serviceName, instanceId, sqsQueue = 'AC-SupportQueue', bucket, redisInstance, region = 'eu-central-1', profile = process.env['profile'], debug } = {}) {
+  init: async function({ serviceName, instanceId, sqsQueue = 'AC-SupportQueue', bucket, redisInstance, region = 'eu-central-1', profile = process.env['profile'], debug, logger = console, throwError = true } = {}) {
     
     this.sqsQueue = sqsQueue
     this.serviceName = serviceName || 'SupportConnector'
@@ -17,6 +17,12 @@ module.exports = {
     this.redisInstance = redisInstance
     this.cache = new NodeCache()
     this.debugMode = debug
+    this.logger = {
+      info:  logger?.info?.bind(logger)  ?? console.info.bind(console),
+      warn:  logger?.warn?.bind(logger)  ?? console.warn.bind(console),
+      error: logger?.error?.bind(logger) ?? console.error.bind(console),
+    }
+    this.throwError = throwError
 
     if (!debug) {
       const client = new STSClient({
@@ -31,7 +37,8 @@ module.exports = {
         availableLists: [{
           name: sqsQueue,
           fifo: true
-        }]
+        }],
+        throwError
       }
       if (profile) { sqsParams.profile = profile } // Optional AWS profile, see below
       if (bucket) {
@@ -61,7 +68,7 @@ module.exports = {
           })
         }
         await this.createMessage(ticket)
-        console.warn(`Test Message ${identifier} sent`)
+        this.logger.warn(`Test Message ${identifier} sent`)
         process.exit(0)
       }
     }
@@ -111,11 +118,14 @@ module.exports = {
     }
     try {
       const response = await this.acsqs.sendSQSMessage({ name: this.sqsQueue, message: JSON.stringify(messagePayload), messageGroupId: (messageGroupId || this.serviceName) })
-      if (this.debugMode) { console.warn(response) }
+      if (this.debugMode) { this.logger.info(response) }
       return response
     }
     catch(e) {
-      console.error('%s | Subject %s | Failed %j', functionName, subject, e?.message)
+      this.logger.error('%s | Subject %s | Failed %j', functionName, subject, e?.message)
+      if (this.throwError) { 
+        throw e
+      }
     }
   }
 }

package/package.json

@@ -3,19 +3,18 @@
   "author": "Mark Poepping (https://www.admiralcloud.com)",
   "license": "MIT",
   "repository": "admiralcloud/ac-support-connector",
-  "version": "1.0.12",
+  "version": "1.0.14",
   "dependencies": {
-    "@aws-sdk/client-sts": "^3.1014.0",
-    "ac-sqs": "^4.0.5",
+    "@aws-sdk/client-sts": "^3.1024.0",
+    "ac-sqs": "^4.0.6",
     "node-cache": "^5.1.2"
   },
   "devDependencies": {
-    "ac-semantic-release": "^0.4.10",
+    "ac-semantic-release": "^1.0.1",
     "chai": "^4.5.0",
-    "eslint": "^10.1.0",
+    "eslint": "^10.2.0",
     "expect": "^30.3.0",
     "globals": "^17.4.0",
-    "ioredis": "^5.10.1",
     "mocha": "^11.7.5"
   },
   "scripts": {

package/test/test.js

@@ -1,11 +1,24 @@
 const { expect } = require('chai');
 
-const Redis = require("ioredis");
-const redisInstance = new Redis() 
-
 const supCon = require('../index')
 
-const string1001 = new Array(512000).join('b')
+// Minimal Redis stub – no real connection needed
+function createFakeRedis() {
+  const store = new Map()
+  return {
+    async set(key, _value, exFlag, seconds, nxFlag) {
+      if (nxFlag === 'nx' && store.has(key)) { return null }
+      store.set(key, true)
+      setTimeout(() => store.delete(key), seconds * 1000)
+      return 'OK'
+    },
+    quit() {}
+  }
+}
+
+const redisInstance = createFakeRedis()
+
+const string1001 = new Array(1100000).join('b')
 
 function timeout(ms) {
   return new Promise(resolve => setTimeout(resolve, ms));
@@ -25,7 +38,7 @@ describe('Basic tests', () => {
     expect(response.message[0]).to.have.property('content', params.text)
   })
 
-  it('Check truncated payload > 256KB - should not throw an error', async() => {
+  it('Check truncated payload > 1MB - should not throw an error', async() => {
     const params = {
       subject: 'Failed',
       message: [{
@@ -140,6 +153,116 @@ describe('Block with Memory', () => {
 })
 
 
+describe('throwError behavior', () => {
+  const originalEnv = process.env.NODE_ENV
+  let fakeLogger
+
+  before(async() => {
+    fakeLogger = { errors: [], warns: [], error(...args) { this.errors.push(args) }, warn(...args) { this.warns.push(args) } }
+    await supCon.init({ debug: true, logger: fakeLogger })
+    process.env.NODE_ENV = 'production' // bypass early return in createMessage
+    supCon.acsqs = {
+      async sendSQSMessage() {
+        throw new Error('SQS failed')
+      }
+    }
+  })
+
+  after(() => {
+    process.env.NODE_ENV = originalEnv
+    delete supCon.acsqs
+  })
+
+  it('throwError: true - should throw on SQS failure', async() => {
+    supCon.throwError = true
+    fakeLogger.errors = []
+    let threw = false
+    try {
+      await supCon.createMessage({ subject: 'Test', text: 'Error' })
+    }
+ catch(e) {
+      threw = true
+      expect(e.message).to.equal('SQS failed')
+    }
+    expect(threw).to.be.true
+    expect(fakeLogger.errors).to.have.lengthOf(1)
+  })
+
+  it('throwError: false - should not throw on SQS failure', async() => {
+    supCon.throwError = false
+    fakeLogger.errors = []
+    const response = await supCon.createMessage({ subject: 'Test', text: 'Error' })
+    expect(response).to.be.undefined
+    expect(fakeLogger.errors).to.have.lengthOf(1)
+  })
+})
+
+describe('custom logger', () => {
+  const originalEnv = process.env.NODE_ENV
+  let fakeLogger
+
+  before(async() => {
+    fakeLogger = { infos: [], info(...args) { this.infos.push(args) } }
+    await supCon.init({ debug: true, logger: fakeLogger })
+    process.env.NODE_ENV = 'production'
+    supCon.acsqs = {
+      async sendSQSMessage() { return { MessageId: 'abc123' } }
+    }
+  })
+
+  after(() => {
+    process.env.NODE_ENV = originalEnv
+    delete supCon.acsqs
+  })
+
+  it('logger.info is called on successful send in debug mode', async() => {
+    const response = await supCon.createMessage({ subject: 'Test', text: 'Hello' })
+    expect(response).to.deep.equal({ MessageId: 'abc123' })
+    expect(fakeLogger.infos).to.have.lengthOf(1)
+  })
+})
+
+
+describe('logger normalization', () => {
+  const originalEnv = process.env.NODE_ENV
+
+  before(() => {
+    process.env.NODE_ENV = 'production'
+  })
+
+  after(() => {
+    process.env.NODE_ENV = originalEnv
+    delete supCon.acsqs
+  })
+
+  it('null logger - falls back to console without throwing', async() => {
+    await supCon.init({ debug: true, logger: null })
+    supCon.acsqs = { async sendSQSMessage() { throw new Error('boom') } }
+    supCon.throwError = false
+    // must not throw a TypeError about this.logger.error
+    await supCon.createMessage({ subject: 'Test', text: 'Hello' })
+  })
+
+  it('partial logger (missing warn/error) - fills gaps without throwing', async() => {
+    const partial = { infos: [], info(...args) { this.infos.push(args) } }
+    await supCon.init({ debug: true, logger: partial })
+    supCon.acsqs = { async sendSQSMessage() { throw new Error('boom') } }
+    supCon.throwError = false
+    // error method is missing on partial — must not throw a TypeError
+    await supCon.createMessage({ subject: 'Test', text: 'Hello' })
+  })
+
+  it('partial logger (missing info) - successful send does not throw', async() => {
+    const partial = { errors: [], error(...args) { this.errors.push(args) } }
+    await supCon.init({ debug: true, logger: partial })
+    supCon.acsqs = { async sendSQSMessage() { return { MessageId: 'xyz' } } }
+    // info method is missing — must not throw a TypeError on success path
+    const response = await supCon.createMessage({ subject: 'Test', text: 'Hello' })
+    expect(response).to.deep.equal({ MessageId: 'xyz' })
+  })
+})
+
+
 describe('Cleanup', () => {
   it('Close Redis', async() => {
     redisInstance.quit()