Merge pull request #12 from bknd-io/chores/remove-deps

Remove unusued deps and fixing auth tests
This commit is contained in:
dswbx
2024-12-03 17:18:18 +01:00
committed by GitHub
4 changed files with 15 additions and 7 deletions

View File

@@ -39,7 +39,10 @@ describe("AppAuth", () => {
test("creates user on register", async () => {
const auth = new AppAuth(
{
enabled: true
enabled: true,
jwt: {
secret: "123456"
}
},
ctx
);
@@ -57,6 +60,9 @@ describe("AppAuth", () => {
disableConsoleLog();
const res = await app.request("/password/register", {
method: "POST",
headers: {
"Content-Type": "application/json"
},
body: JSON.stringify({
email: "some@body.com",
password: "123456"

View File

@@ -45,7 +45,6 @@
"@uiw/react-codemirror": "^4.23.6",
"@xyflow/react": "^12.3.2",
"aws4fetch": "^1.0.18",
"codemirror-lang-liquid": "^1.0.0",
"dayjs": "^1.11.13",
"fast-xml-parser": "^4.4.0",
"hono": "^4.6.12",
@@ -57,12 +56,10 @@
"react-hook-form": "^7.53.1",
"react-icons": "5.2.1",
"react-json-view-lite": "^2.0.1",
"reactflow": "^11.11.4",
"tailwind-merge": "^2.5.4",
"tailwindcss-animate": "^1.0.7",
"wouter": "^3.3.5",
"zod": "^3.23.8",
"zod-to-json-schema": "^3.23.2"
"zod": "^3.23.8"
},
"devDependencies": {
"@aws-sdk/client-s3": "^3.613.0",

View File

@@ -11,7 +11,7 @@ import {
} from "core/utils";
import type { Context, Hono } from "hono";
import { deleteCookie, getSignedCookie, setSignedCookie } from "hono/cookie";
import { decode, sign, verify } from "hono/jwt";
import { sign, verify } from "hono/jwt";
import type { CookieOptions } from "hono/utils/cookie";
import { omit } from "lodash-es";
@@ -177,7 +177,12 @@ export class Authenticator<Strategies extends Record<string, Strategy> = Record<
payload.exp = Math.floor(Date.now() / 1000) + this.config.jwt.expires;
}
return sign(payload, this.config.jwt?.secret ?? "", this.config.jwt?.alg ?? "HS256");
const secret = this.config.jwt.secret;
if (!secret || secret.length === 0) {
throw new Error("Cannot sign JWT without a secret");
}
return sign(payload, secret, this.config.jwt?.alg ?? "HS256");
}
async verify(jwt: string): Promise<boolean> {