fix: update jsonwebtoken from v9.0.1 to v9.0.2

Ref: https://github.blog/2023-04-19-introducing-npm-package-provenance/

.eslintrc.json

@@ -1,22 +1,15 @@
 {
-  "extends": ["standard-with-typescript", "prettier"],
-  "plugins": ["unicorn", "import", "prettier"],
+  "extends": ["conventions", "prettier"],
+  "plugins": ["prettier", "import", "unicorn"],
+  "parser": "@typescript-eslint/parser",
   "parserOptions": {
     "project": "./tsconfig.json"
   },
   "env": {
-    "node": true,
-    "jest": true
+    "node": true
   },
   "rules": {
     "prettier/prettier": "error",
-    "import/order": [
-      "error",
-      {
-        "groups": ["builtin", "external", "internal"],
-        "newlines-between": "always"
-      }
-    ],
     "import/extensions": ["error", "always"],
     "unicorn/prefer-node-protocol": "error"
   }

.github/PULL_REQUEST_TEMPLATE.md

@@ -1,6 +1,6 @@
 <!-- Please first discuss the change you wish to make via issue before making a change. It might avoid a waste of your time. -->
 
-## What changes this PR introduce?
+# What changes this PR introduce?
 
 ## List any relevant issue numbers
 

.github/workflows/build.yml

@@ -2,7 +2,7 @@ name: 'Build'
 
 on:
   push:
-    branches: [master, develop]
+    branches: [develop]
   pull_request:
     branches: [master, develop]
 
@@ -10,16 +10,18 @@ jobs:
   build:
     runs-on: 'ubuntu-latest'
     steps:
-      - uses: 'actions/checkout@v2'
+      - uses: 'actions/checkout@v4.0.0'
 
-      - name: 'Use Node.js'
-        uses: 'actions/setup-node@v2.5.1'
+      - name: 'Setup Node.js'
+        uses: 'actions/setup-node@v3.8.1'
         with:
-          node-version: 'lts/*'
+          node-version: '20.x'
           cache: 'npm'
 
-      - name: 'Install'
-        run: 'npm install'
+      - name: 'Install dependencies'
+        run: 'npm clean-install'
 
-      - name: 'Build Package'
+      - name: 'Build'
         run: 'npm run build'
+
+      - run: 'npm run build:typescript'

.github/workflows/lint.yml

@@ -2,7 +2,7 @@ name: 'Lint'
 
 on:
   push:
-    branches: [master, develop]
+    branches: [develop]
   pull_request:
     branches: [master, develop]
 
@@ -10,19 +10,19 @@ jobs:
   lint:
     runs-on: 'ubuntu-latest'
     steps:
-      - uses: 'actions/checkout@v2'
+      - uses: 'actions/checkout@v4.0.0'
 
-      - name: 'Use Node.js'
-        uses: 'actions/setup-node@v2.5.1'
+      - name: 'Setup Node.js'
+        uses: 'actions/setup-node@v3.8.1'
         with:
-          node-version: 'lts/*'
+          node-version: '20.x'
           cache: 'npm'
 
-      - name: 'Install'
-        run: 'npm install'
+      - name: 'Install dependencies'
+        run: 'npm clean-install'
 
       - run: 'npm run lint:commit -- --to "${{ github.sha }}"'
       - run: 'npm run lint:editorconfig'
       - run: 'npm run lint:markdown'
-      - run: 'npm run lint:typescript'
+      - run: 'npm run lint:eslint'
       - run: 'npm run lint:prettier'

.github/workflows/release.yml

@@ -7,21 +7,31 @@ on:
 jobs:
   release:
     runs-on: 'ubuntu-latest'
+    permissions:
+      contents: 'write'
+      issues: 'write'
+      pull-requests: 'write'
+      id-token: 'write'
     steps:
-      - uses: 'actions/checkout@v2'
+      - uses: 'actions/checkout@v4.0.0'
 
-      - name: 'Use Node.js'
-        uses: 'actions/setup-node@v2.5.1'
+      - name: 'Setup Node.js'
+        uses: 'actions/setup-node@v3.8.1'
         with:
-          node-version: 'lts/*'
+          node-version: '20.x'
           cache: 'npm'
 
-      - name: 'Install'
-        run: 'npm install'
+      - name: 'Install dependencies'
+        run: 'npm clean-install'
 
       - name: 'Build Package'
         run: 'npm run build'
 
+      - run: 'npm run build:typescript'
+
+      - name: 'Verify the integrity of provenance attestations and registry signatures for installed dependencies'
+        run: 'npm audit signatures'
+
       - name: 'Release'
         run: 'npm run release'
         env:

.github/workflows/test.yml

@@ -2,7 +2,7 @@ name: 'Test'
 
 on:
   push:
-    branches: [master, develop]
+    branches: [develop]
   pull_request:
     branches: [master, develop]
 
@@ -10,19 +10,19 @@ jobs:
   test:
     runs-on: 'ubuntu-latest'
     steps:
-      - uses: 'actions/checkout@v2'
+      - uses: 'actions/checkout@v4.0.0'
 
-      - name: 'Use Node.js'
-        uses: 'actions/setup-node@v2.5.1'
+      - name: 'Setup Node.js'
+        uses: 'actions/setup-node@v3.8.1'
         with:
-          node-version: 'lts/*'
+          node-version: '20.x'
           cache: 'npm'
 
-      - name: 'Install'
-        run: 'npm install'
+      - name: 'Install dependencies'
+        run: 'npm clean-install'
+
+      - name: 'Build'
+        run: 'npm run build'
 
       - name: 'Test'
         run: 'npm run test'
-
-      - name: 'Upload Coverage'
-        uses: 'codecov/codecov-action@v2.1.0'

.gitignore

@@ -10,6 +10,7 @@ build
 
 # testing
 coverage
+.nyc_output
 
 # debug
 npm-debug.log*

.husky/pre-commit

@@ -3,3 +3,4 @@
 
 npm run lint:staged
 npm run build
+npm run build:typescript

.lintstagedrc.json

@@ -1,10 +1,6 @@
 {
   "*": ["editorconfig-checker"],
-  "*.{js,jsx,ts,tsx}": [
-    "prettier --write",
-    "eslint --fix",
-    "jest --findRelatedTests"
-  ],
+  "*.{js,jsx,ts,tsx}": ["prettier --write", "eslint --fix"],
   "*.{json,jsonc,yml,yaml}": ["prettier --write"],
-  "*.{md}": ["prettier --write", "markdownlint --dot --fix"]
+  "*.{md,mdx}": ["prettier --write", "markdownlint-cli2 --fix"]
 }

.markdownlint-cli2.jsonc

@@ -0,0 +1,11 @@
+{
+  "config": {
+    "extends": "markdownlint/style/prettier",
+    "relative-links": true,
+    "default": true,
+    "MD033": false
+  },
+  "globs": ["**/*.{md,mdx}"],
+  "ignores": ["**/node_modules"],
+  "customRules": ["markdownlint-rule-relative-links"]
+}

.markdownlint.json

@@ -1,7 +0,0 @@
-{
-  "default": true,
-  "MD013": false,
-  "MD024": false,
-  "MD033": false,
-  "MD041": false
-}

.npmrc

@@ -1 +1,2 @@
 save-exact=true
+provenance=true

.swcrc

@@ -1,22 +1,13 @@
 {
+  "sourceMaps": true,
   "jsc": {
     "parser": {
       "syntax": "typescript",
-      "decorators": true,
       "dynamicImport": true
     },
-    "transform": {
-      "legacyDecorator": true,
-      "decoratorMetadata": true
-    },
-    "target": "es2022",
-    "loose": true
+    "target": "esnext"
   },
   "module": {
-    "type": "commonjs",
-    "strict": false,
-    "strictMode": true,
-    "lazy": false,
-    "noInterop": false
+    "type": "es6"
   }
 }

.vscode/settings.json

@@ -6,5 +6,6 @@
   "editor.formatOnSave": true,
   "editor.codeActionsOnSave": {
     "source.fixAll": true
-  }
+  },
+  "eslint.options": { "ignorePath": ".gitignore" }
 }

CODE_OF_CONDUCT.md

@@ -60,7 +60,7 @@ representative at an online or offline event.
 
 Instances of abusive, harassing, or otherwise unacceptable behavior may be
 reported to the community leaders responsible for enforcement at
-contact@divlo.fr.
+<contact@theoludwig.fr>.
 All complaints will be reviewed and investigated promptly and fairly.
 
 All community leaders are obligated to respect the privacy and security of the

CONTRIBUTING.md

@@ -21,7 +21,7 @@ All work on **Thream** happens directly on [GitHub](https://github.com/Thream).
 
 - **Please first discuss** the change you wish to make via [issue](https://github.com/Thream/socketio-jwt/issues) before making a change. It might avoid a waste of your time.
 
-- Ensure your code respect `eslint` and `prettier`.
+- Ensure your code respect linting.
 
 - Make sure your **code passes the tests**.
 
@@ -29,26 +29,4 @@ If you're adding new features to **Thream/socketio-jwt**, please include tests.
 
 ## Commits
 
-The commit message guidelines respect [@commitlint/config-conventional](https://github.com/conventional-changelog/commitlint/tree/master/%40commitlint/config-conventional) and [Semantic Versioning](https://semver.org/) for releases.
-
-### Types
-
-Types define which kind of changes you made to the project.
-
-| Types    | Description                                                                                                  |
-| -------- | ------------------------------------------------------------------------------------------------------------ |
-| feat     | A new feature.                                                                                               |
-| fix      | A bug fix.                                                                                                   |
-| docs     | Documentation only changes.                                                                                  |
-| style    | Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc).      |
-| refactor | A code change that neither fixes a bug nor adds a feature.                                                   |
-| perf     | A code change that improves performance.                                                                     |
-| test     | Adding missing tests or correcting existing tests.                                                           |
-| build    | Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm).         |
-| ci       | Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs). |
-| chore    | Other changes that don't modify src or test files.                                                           |
-| revert   | Reverts a previous commit.                                                                                   |
-
-### Scopes
-
-Scopes define what part of the code changed.
+The commit message guidelines adheres to [Conventional Commits](https://www.conventionalcommits.org/) and [Semantic Versioning](https://semver.org/) for releases.

LICENSE

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) Auth0, Inc. <support@auth0.com> (http://auth0.com) and Thream contributors
+Copyright (c) Auth0, Inc. <support@auth0.com> (<https://auth0.com/>) and Thream contributors
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

README.md

@@ -12,7 +12,6 @@
   <a href="https://github.com/Thream/socketio-jwt/actions/workflows/build.yml"><img src="https://github.com/Thream/socketio-jwt/actions/workflows/build.yml/badge.svg?branch=develop" /></a>
   <a href="https://github.com/Thream/socketio-jwt/actions/workflows/lint.yml"><img src="https://github.com/Thream/socketio-jwt/actions/workflows/lint.yml/badge.svg?branch=develop" /></a>
   <a href="https://github.com/Thream/socketio-jwt/actions/workflows/test.yml"><img src="https://github.com/Thream/socketio-jwt/actions/workflows/test.yml/badge.svg?branch=develop" /></a>
-  <a href="https://codecov.io/gh/Thream/socketio-jwt"><img src="https://codecov.io/gh/Thream/socketio-jwt/branch/develop/graph/badge.svg" alt="codecov" /></a>
   <br />
   <a href="https://conventionalcommits.org"><img src="https://img.shields.io/badge/Conventional%20Commits-1.0.0-yellow.svg" alt="Conventional Commits" /></a>
   <a href="https://github.com/semantic-release/semantic-release"><img src="https://img.shields.io/badge/%20%20%F0%9F%93%A6%F0%9F%9A%80-semantic--release-e10079.svg" alt="semantic-release" /></a>
@@ -23,9 +22,12 @@
 
 Authenticate socket.io incoming connections with JWTs.
 
-Compatible with `socket.io >= 3.0.0`.
+This repository was originally forked from [auth0-socketio-jwt](https://github.com/auth0-community/auth0-socketio-jwt) and it is not intended to take any credit but to improve the code from now on.
 
-This repository was originally forked from [auth0-socketio-jwt](https://github.com/auth0-community/auth0-socketio-jwt) & it is not intended to take any credit but to improve the code from now on.
+## Prerequisites
+
+- [Node.js](https://nodejs.org/) >= 16.0.0
+- [Socket.IO](https://socket.io/) >= 3.0.0
 
 ## 💾 Install
 
@@ -129,7 +131,7 @@ io.on('connection', async (socket) => {
 
 ```ts
 import { io } from 'socket.io-client'
-import { isUnauthorizedError } from '@thream/socketio-jwt'
+import { isUnauthorizedError } from '@thream/socketio-jwt/build/UnauthorizedError.js'
 
 // Require Bearer Token
 const socket = io('http://localhost:9000', {

jest.config.json

@@ -1,10 +0,0 @@
-{
-  "testEnvironment": "node",
-  "resolver": "jest-ts-webcompat-resolver",
-  "transform": {
-    "^.+\\.(t|j)sx?$": ["@swc/jest"]
-  },
-  "rootDir": "./src",
-  "collectCoverage": true,
-  "coverageDirectory": "../coverage/"
-}

package.json

@@ -1,26 +1,32 @@
 {
   "name": "@thream/socketio-jwt",
   "version": "0.0.0-development",
+  "type": "module",
   "public": true,
   "description": "Authenticate socket.io incoming connections with JWTs.",
   "license": "MIT",
   "main": "build/index.js",
   "types": "build/index.d.ts",
   "files": [
-    "build"
+    "build",
+    "!**/*.test.js",
+    "!**/*.test.d.ts",
+    "!**/*.map"
   ],
   "engines": {
-    "node": ">=12.0.0"
+    "node": ">=16.0.0",
+    "npm": ">=9.0.0"
   },
   "publishConfig": {
-    "access": "public"
+    "access": "public",
+    "provenance": true
   },
   "keywords": [
     "socket",
     "socket.io",
     "jwt"
   ],
-  "author": "Divlo <contact@divlo.fr>",
+  "author": "Théo LUDWIG <contact@theoludwig.fr>",
   "repository": {
     "type": "git",
     "url": "https://github.com/Thream/socketio-jwt"
@@ -30,14 +36,16 @@
   },
   "homepage": "https://github.com/Thream/socketio-jwt#readme",
   "scripts": {
-    "build": "rimraf ./build && swc ./src --out-dir ./build && tsc",
+    "build": "rimraf ./build && swc ./src --out-dir ./build",
+    "build:dev": "swc ./src --out-dir ./build --watch",
+    "build:typescript": "tsc",
     "lint:commit": "commitlint",
     "lint:editorconfig": "editorconfig-checker",
-    "lint:markdown": "markdownlint \"**/*.md\" --dot --ignore-path \".gitignore\"",
-    "lint:typescript": "eslint \"**/*.{js,jsx,ts,tsx}\" --ignore-path \".gitignore\"",
-    "lint:prettier": "prettier \".\" --check --ignore-path \".gitignore\"",
+    "lint:markdown": "markdownlint-cli2",
+    "lint:eslint": "eslint . --max-warnings 0 --report-unused-disable-directives --ignore-path .gitignore",
+    "lint:prettier": "prettier . --check",
     "lint:staged": "lint-staged",
-    "test": "jest",
+    "test": "cross-env NODE_ENV=test node --enable-source-maps --test build/",
     "release": "semantic-release",
     "postinstall": "husky install",
     "prepublishOnly": "pinst --disable",
@@ -47,43 +55,39 @@
     "socket.io": ">=3.0.0"
   },
   "dependencies": {
-    "jsonwebtoken": "8.5.1"
+    "jsonwebtoken": "9.0.2"
   },
   "devDependencies": {
-    "@commitlint/cli": "16.2.1",
-    "@commitlint/config-conventional": "16.2.1",
-    "@swc/cli": "0.1.55",
-    "@swc/core": "1.2.141",
-    "@swc/jest": "0.2.17",
-    "@types/express": "4.17.13",
-    "@types/jest": "27.4.0",
-    "@types/jsonwebtoken": "8.5.8",
-    "@types/node": "17.0.18",
-    "@types/server-destroy": "1.0.1",
-    "@typescript-eslint/eslint-plugin": "4.32.0",
-    "axios": "0.26.0",
-    "editorconfig-checker": "4.0.2",
-    "eslint": "7.32.0",
-    "eslint-config-prettier": "8.3.0",
-    "eslint-config-standard-with-typescript": "21.0.1",
-    "eslint-plugin-import": "2.25.4",
-    "eslint-plugin-node": "11.1.0",
-    "eslint-plugin-prettier": "4.0.0",
-    "eslint-plugin-promise": "5.2.0",
-    "eslint-plugin-unicorn": "40.1.0",
-    "express": "4.17.3",
-    "husky": "7.0.4",
-    "jest": "27.5.1",
-    "jest-ts-webcompat-resolver": "1.0.0",
-    "lint-staged": "12.3.4",
-    "markdownlint-cli": "0.31.1",
-    "pinst": "2.1.6",
-    "rimraf": "3.0.2",
-    "semantic-release": "19.0.2",
-    "server-destroy": "1.0.1",
-    "prettier": "2.5.1",
-    "socket.io": "4.4.1",
-    "socket.io-client": "4.4.1",
-    "typescript": "4.5.5"
+    "@commitlint/cli": "17.7.1",
+    "@commitlint/config-conventional": "17.7.0",
+    "@swc/cli": "0.1.62",
+    "@swc/core": "1.3.85",
+    "@tsconfig/strictest": "2.0.2",
+    "@types/jsonwebtoken": "9.0.3",
+    "@types/node": "20.6.2",
+    "@typescript-eslint/eslint-plugin": "6.7.2",
+    "@typescript-eslint/parser": "6.7.2",
+    "axios": "1.5.0",
+    "cross-env": "7.0.3",
+    "editorconfig-checker": "5.1.1",
+    "eslint": "8.49.0",
+    "eslint-config-conventions": "11.0.1",
+    "eslint-config-prettier": "9.0.0",
+    "eslint-plugin-import": "2.28.1",
+    "eslint-plugin-prettier": "5.0.0",
+    "eslint-plugin-promise": "6.1.1",
+    "eslint-plugin-unicorn": "48.0.1",
+    "fastify": "4.23.2",
+    "husky": "8.0.3",
+    "lint-staged": "14.0.1",
+    "markdownlint-cli2": "0.10.0",
+    "markdownlint-rule-relative-links": "2.1.0",
+    "pinst": "3.0.0",
+    "prettier": "3.0.3",
+    "rimraf": "5.0.1",
+    "semantic-release": "22.0.0",
+    "socket.io": "4.7.2",
+    "socket.io-client": "4.7.2",
+    "typescript": "5.2.2"
   }
 }

src/UnauthorizedError.ts

@@ -4,7 +4,7 @@ export class UnauthorizedError extends Error {
 
   constructor(code: string, error: { message: string }) {
     super(error.message)
-    this.message = error.message
+    this.name = 'UnauthorizedError'
     this.inner = error
     this.data = {
       message: this.message,
@@ -15,6 +15,16 @@ export class UnauthorizedError extends Error {
   }
 }
 
-export const isUnauthorizedError = (error: any): error is UnauthorizedError => {
-  return error.data.type === 'UnauthorizedError'
+export const isUnauthorizedError = (
+  error: unknown
+): error is UnauthorizedError => {
+  return (
+    typeof error === 'object' &&
+    error != null &&
+    'data' in error &&
+    typeof error.data === 'object' &&
+    error.data != null &&
+    'type' in error.data &&
+    error.data.type === 'UnauthorizedError'
+  )
 }

src/__test__/authorize.test.ts

@@ -1,194 +1,217 @@
+import test from 'node:test'
+import assert from 'node:assert/strict'
+
 import axios from 'axios'
+import type { Socket } from 'socket.io-client'
 import { io } from 'socket.io-client'
 
 import { isUnauthorizedError } from '../UnauthorizedError.js'
+import type { Profile } from './fixture/index.js'
 import {
+  API_URL,
   fixtureStart,
   fixtureStop,
   getSocket,
-  Profile
+  basicProfile
 } from './fixture/index.js'
 
-describe('authorize - with secret as string in options', () => {
-  let token: string = ''
-
-  beforeEach((done) => {
-    fixtureStart(async () => {
-      const response = await axios.post('http://localhost:9000/login')
-      token = response.data.token
-    })
-      .then(done)
-      .catch((error) => {
-        done(error)
-      })
-  })
-
-  afterEach((done) => {
-    fixtureStop(done)
-  })
-
-  it('should emit error with no token provided', (done) => {
-    const socket = io('http://localhost:9000')
-    socket.on('connect_error', (error) => {
-      expect(isUnauthorizedError(error)).toBeTruthy()
-      if (isUnauthorizedError(error)) {
-        expect(error.data.message).toEqual('no token provided')
-        expect(error.data.code).toEqual('credentials_required')
-      }
-      socket.close()
-      done()
-    })
-  })
-
-  it('should emit error with bad token format', (done) => {
-    const socket = io('http://localhost:9000', {
-      auth: { token: 'testing' }
-    })
-    socket.on('connect_error', (error) => {
-      expect(isUnauthorizedError(error)).toBeTruthy()
-      if (isUnauthorizedError(error)) {
-        expect(error.data.message).toEqual(
-          'Format is Authorization: Bearer [token]'
-        )
-        expect(error.data.code).toEqual('credentials_bad_format')
-      }
-      socket.close()
-      done()
-    })
-  })
-
-  it('should emit error with unauthorized handshake', (done) => {
-    const socket = io('http://localhost:9000', {
-      auth: { token: 'Bearer testing' }
-    })
-    socket.on('connect_error', (error) => {
-      expect(isUnauthorizedError(error)).toBeTruthy()
-      if (isUnauthorizedError(error)) {
-        expect(error.data.message).toEqual(
-          'Unauthorized: Token is missing or invalid Bearer'
-        )
-        expect(error.data.code).toEqual('invalid_token')
-      }
-      socket.close()
-      done()
-    })
-  })
-
-  it('should connect the user', (done) => {
-    const socket = io('http://localhost:9000', {
-      auth: { token: `Bearer ${token}` }
-    })
-    socket.on('connect', () => {
-      socket.close()
-      done()
-    })
-    socket.on('connect_error', (error) => {
-      done(error)
-    })
-  })
+export const api = axios.create({
+  baseURL: API_URL,
+  headers: {
+    'Content-Type': 'application/json'
+  }
 })
 
 const secretCallback = async (): Promise<string> => {
   return 'somesecret'
 }
 
-describe('authorize - with secret as callback in options', () => {
-  let token: string = ''
+await test('authorize', async (t) => {
+  await t.test('with secret as string in options', async (t) => {
+    let token = ''
+    let socket: Socket | null = null
 
-  beforeEach((done) => {
-    fixtureStart(
-      async () => {
-        const response = await axios.post('http://localhost:9000/login')
-        token = response.data.token
-      },
-      { secret: secretCallback }
-    )
-      .then(done)
-      .catch((error) => {
-        done(error)
+    t.beforeEach(async () => {
+      await fixtureStart()
+      const response = await api.post('/login', {})
+      token = response.data.token
+    })
+
+    t.afterEach(async () => {
+      socket?.disconnect()
+      await fixtureStop()
+    })
+
+    await t.test('should emit error with no token provided', () => {
+      socket = io(API_URL)
+      socket.on('connect_error', async (error) => {
+        assert.strictEqual(isUnauthorizedError(error), true)
+        if (isUnauthorizedError(error)) {
+          assert.strictEqual(error.data.message, 'no token provided')
+          assert.strictEqual(error.data.code, 'credentials_required')
+          assert.ok(true)
+        } else {
+          assert.fail('should be unauthorized error')
+        }
       })
-  })
+      socket.on('connect', async () => {
+        assert.fail('should not connect')
+      })
+    })
 
-  afterEach((done) => {
-    fixtureStop(done)
-  })
+    await t.test('should emit error with bad token format', () => {
+      socket = io(API_URL, {
+        auth: { token: 'testing' }
+      })
+      socket.on('connect_error', async (error) => {
+        assert.strictEqual(isUnauthorizedError(error), true)
+        if (isUnauthorizedError(error)) {
+          assert.strictEqual(
+            error.data.message,
+            'Format is Authorization: Bearer [token]'
+          )
+          assert.strictEqual(error.data.code, 'credentials_bad_format')
+          assert.ok(true)
+        } else {
+          assert.fail('should be unauthorized error')
+        }
+      })
+      socket.on('connect', async () => {
+        assert.fail('should not connect')
+      })
+    })
 
-  it('should emit error with no token provided', (done) => {
-    const socket = io('http://localhost:9000')
-    socket.on('connect_error', (error) => {
-      expect(isUnauthorizedError(error)).toBeTruthy()
-      if (isUnauthorizedError(error)) {
-        expect(error.data.message).toEqual('no token provided')
-        expect(error.data.code).toEqual('credentials_required')
-      }
-      socket.close()
-      done()
+    await t.test('should emit error with unauthorized handshake', () => {
+      socket = io(API_URL, {
+        auth: { token: 'Bearer testing' }
+      })
+      socket.on('connect_error', async (error) => {
+        assert.strictEqual(isUnauthorizedError(error), true)
+        if (isUnauthorizedError(error)) {
+          assert.strictEqual(
+            error.data.message,
+            'Unauthorized: Token is missing or invalid Bearer'
+          )
+          assert.strictEqual(error.data.code, 'invalid_token')
+          assert.ok(true)
+        } else {
+          assert.fail('should be unauthorized error')
+        }
+      })
+      socket.on('connect', async () => {
+        assert.fail('should not connect')
+      })
+    })
+
+    await t.test('should connect the user', () => {
+      socket = io(API_URL, {
+        auth: { token: `Bearer ${token}` }
+      })
+      socket.on('connect', async () => {
+        assert.ok(true)
+      })
+      socket.on('connect_error', async (error) => {
+        assert.fail(error.message)
+      })
     })
   })
 
-  it('should emit error with bad token format', (done) => {
-    const socket = io('http://localhost:9000', {
-      auth: { token: 'testing' }
+  await t.test('with secret as callback in options', async (t) => {
+    let token = ''
+    let socket: Socket | null = null
+
+    t.beforeEach(async () => {
+      await fixtureStart({ secret: secretCallback })
+      const response = await api.post('/login', {})
+      token = response.data.token
     })
-    socket.on('connect_error', (error) => {
-      expect(isUnauthorizedError(error)).toBeTruthy()
-      if (isUnauthorizedError(error)) {
-        expect(error.data.message).toEqual(
-          'Format is Authorization: Bearer [token]'
-        )
-        expect(error.data.code).toEqual('credentials_bad_format')
-      }
-      socket.close()
-      done()
+
+    t.afterEach(async () => {
+      socket?.disconnect()
+      await fixtureStop()
+    })
+
+    await t.test('should emit error with no token provided', () => {
+      socket = io(API_URL)
+      socket.on('connect_error', async (error) => {
+        assert.strictEqual(isUnauthorizedError(error), true)
+        if (isUnauthorizedError(error)) {
+          assert.strictEqual(error.data.message, 'no token provided')
+          assert.strictEqual(error.data.code, 'credentials_required')
+          assert.ok(true)
+        } else {
+          assert.fail('should be unauthorized error')
+        }
+      })
+      socket.on('connect', async () => {
+        assert.fail('should not connect')
+      })
+    })
+
+    await t.test('should emit error with bad token format', () => {
+      socket = io(API_URL, {
+        auth: { token: 'testing' }
+      })
+      socket.on('connect_error', async (error) => {
+        assert.strictEqual(isUnauthorizedError(error), true)
+        if (isUnauthorizedError(error)) {
+          assert.strictEqual(
+            error.data.message,
+            'Format is Authorization: Bearer [token]'
+          )
+          assert.strictEqual(error.data.code, 'credentials_bad_format')
+          assert.ok(true)
+        } else {
+          assert.fail('should be unauthorized error')
+        }
+      })
+      socket.on('connect', async () => {
+        assert.fail('should not connect')
+      })
+    })
+
+    await t.test('should emit error with unauthorized handshake', () => {
+      socket = io(API_URL, {
+        auth: { token: 'Bearer testing' }
+      })
+      socket.on('connect_error', async (error) => {
+        assert.strictEqual(isUnauthorizedError(error), true)
+        if (isUnauthorizedError(error)) {
+          assert.strictEqual(
+            error.data.message,
+            'Unauthorized: Token is missing or invalid Bearer'
+          )
+          assert.strictEqual(error.data.code, 'invalid_token')
+          assert.ok(true)
+        } else {
+          assert.fail('should be unauthorized error')
+        }
+      })
+      socket.on('connect', async () => {
+        assert.fail('should not connect')
+      })
+    })
+
+    await t.test('should connect the user', () => {
+      socket = io(API_URL, {
+        auth: { token: `Bearer ${token}` }
+      })
+      socket.on('connect', async () => {
+        assert.ok(true)
+      })
+      socket.on('connect_error', async (error) => {
+        assert.fail(error.message)
+      })
     })
   })
 
-  it('should emit error with unauthorized handshake', (done) => {
-    const socket = io('http://localhost:9000', {
-      auth: { token: 'Bearer testing' }
-    })
-    socket.on('connect_error', (error) => {
-      expect(isUnauthorizedError(error)).toBeTruthy()
-      if (isUnauthorizedError(error)) {
-        expect(error.data.message).toEqual(
-          'Unauthorized: Token is missing or invalid Bearer'
-        )
-        expect(error.data.code).toEqual('invalid_token')
-      }
-      socket.close()
-      done()
-    })
-  })
+  await t.test('with onAuthentication callback in options', async (t) => {
+    let token = ''
+    let wrongToken = ''
+    let socket: Socket | null = null
 
-  it('should connect the user', (done) => {
-    const socket = io('http://localhost:9000', {
-      auth: { token: `Bearer ${token}` }
-    })
-    socket.on('connect', () => {
-      socket.close()
-      done()
-    })
-    socket.on('connect_error', (error) => {
-      done(error)
-    })
-  })
-})
-
-describe('authorize - with onAuthentication callback in options', () => {
-  let token: string = ''
-  let wrongToken: string = ''
-
-  beforeEach((done) => {
-    fixtureStart(
-      async () => {
-        const response = await axios.post('http://localhost:9000/login')
-        token = response.data.token
-        const responseWrong = await axios.post(
-          'http://localhost:9000/login-wrong'
-        )
-        wrongToken = responseWrong.data.token
-      },
-      {
+    t.beforeEach(async () => {
+      await fixtureStart({
         secret: secretCallback,
         onAuthentication: (decodedToken: Profile) => {
           if (!decodedToken.checkField) {
@@ -198,102 +221,120 @@ describe('authorize - with onAuthentication callback in options', () => {
             email: decodedToken.email
           }
         }
-      }
-    )
-      .then(done)
-      .catch((error) => {
-        done(error)
       })
-  })
+      const response = await api.post('/login', {})
+      token = response.data.token
+      const responseWrong = await api.post('/login-wrong', {})
+      wrongToken = responseWrong.data.token
+    })
 
-  afterEach((done) => {
-    fixtureStop(done)
-  })
+    t.afterEach(async () => {
+      socket?.disconnect()
+      await fixtureStop()
+    })
 
-  it('should emit error with no token provided', (done) => {
-    const socket = io('http://localhost:9000')
-    socket.on('connect_error', (error) => {
-      expect(isUnauthorizedError(error)).toBeTruthy()
-      if (isUnauthorizedError(error)) {
-        expect(error.data.message).toEqual('no token provided')
-        expect(error.data.code).toEqual('credentials_required')
-      }
-      socket.close()
-      done()
+    await t.test('should emit error with no token provided', () => {
+      socket = io(API_URL)
+      socket.on('connect_error', async (error) => {
+        assert.strictEqual(isUnauthorizedError(error), true)
+        if (isUnauthorizedError(error)) {
+          assert.strictEqual(error.data.message, 'no token provided')
+          assert.strictEqual(error.data.code, 'credentials_required')
+          assert.ok(true)
+        } else {
+          assert.fail('should be unauthorized error')
+        }
+      })
+      socket.on('connect', async () => {
+        assert.fail('should not connect')
+      })
     })
-  })
 
-  it('should emit error with bad token format', (done) => {
-    const socket = io('http://localhost:9000', {
-      auth: { token: 'testing' }
+    await t.test('should emit error with bad token format', () => {
+      socket = io(API_URL, {
+        auth: { token: 'testing' }
+      })
+      socket.on('connect_error', async (error) => {
+        assert.strictEqual(isUnauthorizedError(error), true)
+        if (isUnauthorizedError(error)) {
+          assert.strictEqual(
+            error.data.message,
+            'Format is Authorization: Bearer [token]'
+          )
+          assert.strictEqual(error.data.code, 'credentials_bad_format')
+          assert.ok(true)
+        } else {
+          assert.fail('should be unauthorized error')
+        }
+      })
+      socket.on('connect', async () => {
+        assert.fail('should not connect')
+      })
     })
-    socket.on('connect_error', (error) => {
-      expect(isUnauthorizedError(error)).toBeTruthy()
-      if (isUnauthorizedError(error)) {
-        expect(error.data.message).toEqual(
-          'Format is Authorization: Bearer [token]'
-        )
-        expect(error.data.code).toEqual('credentials_bad_format')
-      }
-      socket.close()
-      done()
-    })
-  })
 
-  it('should emit error with unauthorized handshake', (done) => {
-    const socket = io('http://localhost:9000', {
-      auth: { token: 'Bearer testing' }
+    await t.test('should emit error with unauthorized handshake', () => {
+      socket = io(API_URL, {
+        auth: { token: 'Bearer testing' }
+      })
+      socket.on('connect_error', async (error) => {
+        assert.strictEqual(isUnauthorizedError(error), true)
+        if (isUnauthorizedError(error)) {
+          assert.strictEqual(
+            error.data.message,
+            'Unauthorized: Token is missing or invalid Bearer'
+          )
+          assert.strictEqual(error.data.code, 'invalid_token')
+          assert.ok(true)
+        } else {
+          assert.fail('should be unauthorized error')
+        }
+      })
+      socket.on('connect', async () => {
+        assert.fail('should not connect')
+      })
     })
-    socket.on('connect_error', (error) => {
-      expect(isUnauthorizedError(error)).toBeTruthy()
-      if (isUnauthorizedError(error)) {
-        expect(error.data.message).toEqual(
-          'Unauthorized: Token is missing or invalid Bearer'
-        )
-        expect(error.data.code).toEqual('invalid_token')
-      }
-      socket.close()
-      done()
-    })
-  })
 
-  it('should connect the user', (done) => {
-    const socket = io('http://localhost:9000', {
-      auth: { token: `Bearer ${token}` }
+    await t.test('should connect the user', () => {
+      socket = io(API_URL, {
+        auth: { token: `Bearer ${token}` }
+      })
+      socket.on('connect', async () => {
+        assert.ok(true)
+      })
+      socket.on('connect_error', async (error) => {
+        assert.fail(error.message)
+      })
     })
-    socket.on('connect', () => {
-      socket.close()
-      done()
-    })
-  })
 
-  it('should contain user property', (done) => {
-    const socketServer = getSocket()
-    socketServer?.on('connection', (client: any) => {
-      expect(client.user.email).toEqual('john@doe.com')
+    await t.test('should contains user properties', () => {
+      const socketServer = getSocket()
+      socketServer?.on('connection', (client: any) => {
+        assert.strictEqual(client.user.email, basicProfile.email)
+        assert.ok(true)
+      })
+      socket = io(API_URL, {
+        auth: { token: `Bearer ${token}` }
+      })
+      socket.on('connect_error', async (error) => {
+        assert.fail(error.message)
+      })
     })
-    const socket = io('http://localhost:9000', {
-      auth: { token: `Bearer ${token}` }
-    })
-    socket.on('connect', () => {
-      socket.close()
-      done()
-    })
-  })
 
-  it('should emit error when user validation fails', (done) => {
-    const socket = io('http://localhost:9000', {
-      auth: { token: `Bearer ${wrongToken}` }
-    })
-    socket.on('connect_error', (err: any) => {
-      try {
-        expect(err.message).toEqual('Check Field validation failed')
-      } catch (err) {
-        socket.close()
-        done(err)
-      }
-      socket.close()
-      done()
+    await t.test('should emit error when user validation fails', () => {
+      socket = io(API_URL, {
+        auth: { token: `Bearer ${wrongToken}` }
+      })
+      socket.on('connect_error', async (error) => {
+        try {
+          assert.strictEqual(error.message, 'Check Field validation failed')
+          assert.ok(true)
+        } catch {
+          assert.fail(error.message)
+        }
+      })
+      socket.on('connect', async () => {
+        assert.fail('should not connect')
+      })
     })
   })
 })

src/__test__/fixture/index.ts

@@ -1,42 +1,43 @@
-import type { Server as HttpServer } from 'node:http'
-import type { Server as HttpsServer } from 'node:https'
-
-import express from 'express'
 import jwt from 'jsonwebtoken'
 import { Server as SocketIoServer } from 'socket.io'
-import enableDestroy from 'server-destroy'
+import type { FastifyInstance } from 'fastify'
+import fastify from 'fastify'
 
-import { authorize, AuthorizeOptions } from '../../index.js'
+import type { AuthorizeOptions } from '../../index.js'
+import { authorize } from '../../index.js'
 
-export interface Profile {
+interface FastifyIo {
+  instance: SocketIoServer
+}
+
+declare module 'fastify' {
+  export interface FastifyInstance {
+    io: FastifyIo
+  }
+}
+
+export interface BasicProfile {
   email: string
   id: number
+}
+
+export interface Profile extends BasicProfile {
   checkField: boolean
 }
 
-interface Socket {
-  io: null | SocketIoServer
-  init: (httpServer: HttpServer | HttpsServer) => void
+export const PORT = 9000
+export const API_URL = `http://localhost:${PORT}`
+export const basicProfile: BasicProfile = {
+  email: 'john@doe.com',
+  id: 123
 }
 
-const socket: Socket = {
-  io: null,
-  init(httpServer) {
-    socket.io = new SocketIoServer(httpServer)
-  }
-}
-
-let server: HttpServer | null = null
+let application: FastifyInstance | null = null
 
 export const fixtureStart = async (
-  done: any,
   options: AuthorizeOptions = { secret: 'super secret' }
 ): Promise<void> => {
-  const profile: Profile = {
-    email: 'john@doe.com',
-    id: 123,
-    checkField: true
-  }
+  const profile: Profile = { ...basicProfile, checkField: true }
   let keySecret = ''
   if (typeof options.secret === 'string') {
     keySecret = options.secret
@@ -46,35 +47,37 @@ export const fixtureStart = async (
       payload: profile
     })
   }
-  const app = express()
-  app.use(express.json())
-  app.post('/login', (_req, res) => {
+  application = fastify()
+  application.post('/login', async (_request, reply) => {
     const token = jwt.sign(profile, keySecret, {
       expiresIn: 60 * 60 * 5
     })
-    return res.json({ token })
+    reply.statusCode = 201
+    return { token }
   })
-  app.post('/login-wrong', (_req, res) => {
+  application.post('/login-wrong', async (_request, reply) => {
     profile.checkField = false
     const token = jwt.sign(profile, keySecret, {
       expiresIn: 60 * 60 * 5
     })
-    return res.json({ token })
+    reply.statusCode = 201
+    return { token }
+  })
+  const instance = new SocketIoServer(application.server)
+  instance.use(authorize(options))
+  application.decorate('io', { instance })
+  application.addHook('onClose', (fastify) => {
+    fastify.io.instance.close()
+  })
+  await application.listen({
+    port: PORT
   })
-  server = app.listen(9000, done)
-  socket.init(server)
-  socket.io?.use(authorize(options))
-  enableDestroy(server)
 }
 
-export const fixtureStop = (callback: Function): void => {
-  socket.io?.close()
-  try {
-    server?.destroy()
-  } catch {}
-  callback()
+export const fixtureStop = async (): Promise<void> => {
+  await application?.close()
 }
 
-export const getSocket = (): SocketIoServer | null => {
-  return socket.io
+export const getSocket = (): SocketIoServer | undefined => {
+  return application?.io.instance
 }

src/authorize.ts

@@ -1,5 +1,6 @@
-import jwt, { Algorithm } from 'jsonwebtoken'
-import { Socket } from 'socket.io'
+import type { Algorithm } from 'jsonwebtoken'
+import jwt from 'jsonwebtoken'
+import type { Socket } from 'socket.io'
 
 import { UnauthorizedError } from './UnauthorizedError.js'
 
@@ -61,7 +62,7 @@ export const authorize = (options: AuthorizeOptions): SocketIOMiddleware => {
     }
     socket.encodedToken = encodedToken
     let keySecret: string | null = null
-    let decodedToken: any
+    let decodedToken: any = null
     if (typeof secret === 'string') {
       keySecret = secret
     } else {

tsconfig.json

@@ -1,15 +1,13 @@
 {
+  "extends": "@tsconfig/strictest/tsconfig.json",
   "compilerOptions": {
     "target": "ESNext",
-    "module": "commonjs",
     "lib": ["ESNext"],
-    "moduleResolution": "node",
+    "module": "NodeNext",
+    "moduleResolution": "NodeNext",
     "outDir": "./build",
     "rootDir": "./src",
     "emitDeclarationOnly": true,
-    "declaration": true,
-    "strict": true,
-    "skipLibCheck": true,
-    "esModuleInterop": true
+    "declaration": true
   }
 }