fix: safer isUnauthorizedError type guard

This commit is contained in:
Divlo 2023-04-02 23:04:41 +02:00
parent 03e8d51f9a
commit 71e0d82655
No known key found for this signature in database
GPG Key ID: 8F9478F220CE65E9
8 changed files with 3164 additions and 1621 deletions

View File

@ -10,7 +10,7 @@ jobs:
build: build:
runs-on: 'ubuntu-latest' runs-on: 'ubuntu-latest'
steps: steps:
- uses: 'actions/checkout@v3.3.0' - uses: 'actions/checkout@v3.5.0'
- name: 'Use Node.js' - name: 'Use Node.js'
uses: 'actions/setup-node@v3.6.0' uses: 'actions/setup-node@v3.6.0'

View File

@ -10,7 +10,7 @@ jobs:
lint: lint:
runs-on: 'ubuntu-latest' runs-on: 'ubuntu-latest'
steps: steps:
- uses: 'actions/checkout@v3.3.0' - uses: 'actions/checkout@v3.5.0'
- name: 'Use Node.js' - name: 'Use Node.js'
uses: 'actions/setup-node@v3.6.0' uses: 'actions/setup-node@v3.6.0'

View File

@ -8,7 +8,7 @@ jobs:
release: release:
runs-on: 'ubuntu-latest' runs-on: 'ubuntu-latest'
steps: steps:
- uses: 'actions/checkout@v3.3.0' - uses: 'actions/checkout@v3.5.0'
- name: 'Use Node.js' - name: 'Use Node.js'
uses: 'actions/setup-node@v3.6.0' uses: 'actions/setup-node@v3.6.0'

View File

@ -10,7 +10,7 @@ jobs:
test: test:
runs-on: 'ubuntu-latest' runs-on: 'ubuntu-latest'
steps: steps:
- uses: 'actions/checkout@v3.3.0' - uses: 'actions/checkout@v3.5.0'
- name: 'Use Node.js' - name: 'Use Node.js'
uses: 'actions/setup-node@v3.6.0' uses: 'actions/setup-node@v3.6.0'

4704
package-lock.json generated

File diff suppressed because it is too large Load Diff

View File

@ -53,38 +53,38 @@
"jsonwebtoken": "9.0.0" "jsonwebtoken": "9.0.0"
}, },
"devDependencies": { "devDependencies": {
"@commitlint/cli": "17.4.1", "@commitlint/cli": "17.5.1",
"@commitlint/config-conventional": "17.4.0", "@commitlint/config-conventional": "17.4.4",
"@swc/cli": "0.1.57", "@swc/cli": "0.1.62",
"@swc/core": "1.3.25", "@swc/core": "1.3.44",
"@tsconfig/strictest": "1.0.2", "@tsconfig/strictest": "2.0.0",
"@types/jsonwebtoken": "9.0.0", "@types/jsonwebtoken": "9.0.1",
"@types/node": "18.11.18", "@types/node": "18.15.11",
"@types/tap": "15.0.7", "@types/tap": "15.0.8",
"@typescript-eslint/eslint-plugin": "5.48.1", "@typescript-eslint/eslint-plugin": "5.57.0",
"@typescript-eslint/parser": "5.48.1", "@typescript-eslint/parser": "5.57.0",
"axios": "1.2.2", "axios": "1.3.4",
"c8": "7.12.0", "c8": "7.13.0",
"editorconfig-checker": "4.0.2", "editorconfig-checker": "5.0.1",
"eslint": "8.31.0", "eslint": "8.37.0",
"eslint-config-conventions": "6.0.0", "eslint-config-conventions": "8.0.0",
"eslint-config-prettier": "8.6.0", "eslint-config-prettier": "8.8.0",
"eslint-plugin-import": "2.26.0", "eslint-plugin-import": "2.27.5",
"eslint-plugin-prettier": "4.2.1", "eslint-plugin-prettier": "4.2.1",
"eslint-plugin-promise": "6.1.1", "eslint-plugin-promise": "6.1.1",
"eslint-plugin-unicorn": "45.0.2", "eslint-plugin-unicorn": "46.0.0",
"fastify": "4.11.0", "fastify": "4.15.0",
"husky": "8.0.3", "husky": "8.0.3",
"lint-staged": "13.1.0", "lint-staged": "13.2.0",
"markdownlint-cli2": "0.6.0", "markdownlint-cli2": "0.6.0",
"markdownlint-rule-relative-links": "1.1.1", "markdownlint-rule-relative-links": "1.1.2",
"pinst": "3.0.0", "pinst": "3.0.0",
"prettier": "2.8.2", "prettier": "2.8.7",
"rimraf": "3.0.2", "rimraf": "4.4.1",
"semantic-release": "20.0.2", "semantic-release": "21.0.1",
"socket.io": "4.5.4", "socket.io": "4.6.1",
"socket.io-client": "4.5.4", "socket.io-client": "4.6.1",
"tap": "16.3.3", "tap": "16.3.4",
"typescript": "4.9.4" "typescript": "5.0.3"
} }
} }

View File

@ -15,6 +15,11 @@ export class UnauthorizedError extends Error {
} }
} }
export const isUnauthorizedError = (error: any): error is UnauthorizedError => { export const isUnauthorizedError = (
return error.data.type === 'UnauthorizedError' error: unknown
): error is UnauthorizedError => {
return (
error instanceof UnauthorizedError &&
error.data.type === 'UnauthorizedError'
)
} }

View File

@ -62,7 +62,7 @@ export const authorize = (options: AuthorizeOptions): SocketIOMiddleware => {
} }
socket.encodedToken = encodedToken socket.encodedToken = encodedToken
let keySecret: string | null = null let keySecret: string | null = null
let decodedToken: any let decodedToken: any = null
if (typeof secret === 'string') { if (typeof secret === 'string') {
keySecret = secret keySecret = secret
} else { } else {