From 5d452a15258a0ec512c56cbe69dfac1b8db03db8 Mon Sep 17 00:00:00 2001 From: Elian Doran Date: Wed, 3 Apr 2024 19:22:49 +0300 Subject: [PATCH] server-ts: Address review --- src/becca/entities/abstract_becca_entity.ts | 2 +- src/becca/similarity.ts | 2 +- src/services/auth.ts | 26 ++++++++++----------- src/services/date_utils.ts | 2 +- src/services/etapi_tokens.ts | 4 ++-- src/services/keyboard_actions.ts | 2 +- src/services/notes.ts | 2 +- src/services/sql.ts | 2 +- 8 files changed, 20 insertions(+), 22 deletions(-) diff --git a/src/becca/entities/abstract_becca_entity.ts b/src/becca/entities/abstract_becca_entity.ts index 00bf78e80..ed9901b2f 100644 --- a/src/becca/entities/abstract_becca_entity.ts +++ b/src/becca/entities/abstract_becca_entity.ts @@ -11,7 +11,7 @@ import protectedSessionService = require('../../services/protected_session'); import blobService = require('../../services/blob'); import Becca, { ConstructorData } from '../becca-interface'; -let becca: Becca | null = null; +let becca: Becca; interface ContentOpts { forceSave?: boolean; diff --git a/src/becca/similarity.ts b/src/becca/similarity.ts index 600e6dc2b..e6721d0df 100644 --- a/src/becca/similarity.ts +++ b/src/becca/similarity.ts @@ -153,7 +153,7 @@ function buildRewardMap(note: BNote) { const mimeCache: Record = {}; -function trimMime(mime?: string) { +function trimMime(mime: string) { if (!mime || mime === 'text/html') { return; } diff --git a/src/services/auth.ts b/src/services/auth.ts index 30f5687f3..0e96f75cb 100644 --- a/src/services/auth.ts +++ b/src/services/auth.ts @@ -7,13 +7,11 @@ import utils = require('./utils'); import passwordEncryptionService = require('./encryption/password_encryption'); import config = require('./config'); import passwordService = require('./encryption/password'); +import type { Request } from 'express'; const noAuthentication = config.General && config.General.noAuthentication === true; -// TODO: We are using custom types for request & response because couldn't extract those pesky express types. -interface Request { - method: string; - path: string; +interface AppRequest extends Request { headers: { authorization?: string; "trilium-cred"?: string; @@ -30,7 +28,7 @@ interface Response { type Callback = () => void; -function checkAuth(req: Request, res: Response, next: Callback) { +function checkAuth(req: AppRequest, res: Response, next: Callback) { if (!sqlInit.isDbInitialized()) { res.redirect("setup"); } @@ -44,7 +42,7 @@ function checkAuth(req: Request, res: Response, next: Callback) { // for electron things which need network stuff // currently, we're doing that for file upload because handling form data seems to be difficult -function checkApiAuthOrElectron(req: Request, res: Response, next: Callback) { +function checkApiAuthOrElectron(req: AppRequest, res: Response, next: Callback) { if (!req.session.loggedIn && !utils.isElectron() && !noAuthentication) { reject(req, res, "Logged in session not found"); } @@ -53,7 +51,7 @@ function checkApiAuthOrElectron(req: Request, res: Response, next: Callback) { } } -function checkApiAuth(req: Request, res: Response, next: Callback) { +function checkApiAuth(req: AppRequest, res: Response, next: Callback) { if (!req.session.loggedIn && !noAuthentication) { reject(req, res, "Logged in session not found"); } @@ -62,7 +60,7 @@ function checkApiAuth(req: Request, res: Response, next: Callback) { } } -function checkAppInitialized(req: Request, res: Response, next: Callback) { +function checkAppInitialized(req: AppRequest, res: Response, next: Callback) { if (!sqlInit.isDbInitialized()) { res.redirect("setup"); } @@ -71,7 +69,7 @@ function checkAppInitialized(req: Request, res: Response, next: Callback) { } } -function checkPasswordSet(req: Request, res: Response, next: Callback) { +function checkPasswordSet(req: AppRequest, res: Response, next: Callback) { if (!utils.isElectron() && !passwordService.isPasswordSet()) { res.redirect("set-password"); } else { @@ -79,7 +77,7 @@ function checkPasswordSet(req: Request, res: Response, next: Callback) { } } -function checkPasswordNotSet(req: Request, res: Response, next: Callback) { +function checkPasswordNotSet(req: AppRequest, res: Response, next: Callback) { if (!utils.isElectron() && passwordService.isPasswordSet()) { res.redirect("login"); } else { @@ -87,7 +85,7 @@ function checkPasswordNotSet(req: Request, res: Response, next: Callback) { } } -function checkAppNotInitialized(req: Request, res: Response, next: Callback) { +function checkAppNotInitialized(req: AppRequest, res: Response, next: Callback) { if (sqlInit.isDbInitialized()) { reject(req, res, "App already initialized."); } @@ -96,7 +94,7 @@ function checkAppNotInitialized(req: Request, res: Response, next: Callback) { } } -function checkEtapiToken(req: Request, res: Response, next: Callback) { +function checkEtapiToken(req: AppRequest, res: Response, next: Callback) { if (etapiTokenService.isValidAuthHeader(req.headers.authorization)) { next(); } @@ -105,7 +103,7 @@ function checkEtapiToken(req: Request, res: Response, next: Callback) { } } -function reject(req: Request, res: Response, message: string) { +function reject(req: AppRequest, res: Response, message: string) { log.info(`${req.method} ${req.path} rejected with 401 ${message}`); res.setHeader("Content-Type", "text/plain") @@ -113,7 +111,7 @@ function reject(req: Request, res: Response, message: string) { .send(message); } -function checkCredentials(req: Request, res: Response, next: Callback) { +function checkCredentials(req: AppRequest, res: Response, next: Callback) { if (!sqlInit.isDbInitialized()) { res.setHeader("Content-Type", "text/plain") .status(400) diff --git a/src/services/date_utils.ts b/src/services/date_utils.ts index 82347504c..88b6ecb69 100644 --- a/src/services/date_utils.ts +++ b/src/services/date_utils.ts @@ -80,7 +80,7 @@ function validateLocalDateTime(str: string | null | undefined) { } } -function validateUtcDateTime(str?: string) { +function validateUtcDateTime(str: string | undefined) { if (!str) { return; } diff --git a/src/services/etapi_tokens.ts b/src/services/etapi_tokens.ts index 46e9c65df..2989d3923 100644 --- a/src/services/etapi_tokens.ts +++ b/src/services/etapi_tokens.ts @@ -25,7 +25,7 @@ function createToken(tokenName: string) { }; } -function parseAuthToken(auth?: string | null) { +function parseAuthToken(auth: string | undefined) { if (!auth) { return null; } @@ -64,7 +64,7 @@ function parseAuthToken(auth?: string | null) { } } -function isValidAuthHeader(auth?: string | null) { +function isValidAuthHeader(auth: string | undefined) { const parsed = parseAuthToken(auth); if (!parsed) { diff --git a/src/services/keyboard_actions.ts b/src/services/keyboard_actions.ts index 8bd830cfe..b2cbaad29 100644 --- a/src/services/keyboard_actions.ts +++ b/src/services/keyboard_actions.ts @@ -623,7 +623,7 @@ function getKeyboardActions() { for (const option of optionService.getOptions()) { if (option.name.startsWith('keyboardShortcuts')) { - let actionName = option.name.substr(17); + let actionName = option.name.substring(17); actionName = actionName.charAt(0).toLowerCase() + actionName.slice(1); const action = actions.find(ea => ea.actionName === actionName); diff --git a/src/services/notes.ts b/src/services/notes.ts index d6efbf31b..a402a2f16 100644 --- a/src/services/notes.ts +++ b/src/services/notes.ts @@ -873,7 +873,7 @@ function getUndeletedParentBranchIds(noteId: string, deleteId: string) { AND parentNote.isDeleted = 0`, [noteId, deleteId]); } -function scanForLinks(note: BNote | null, content: string) { +function scanForLinks(note: BNote, content: string) { if (!note || !['text', 'relationMap'].includes(note.type)) { return; } diff --git a/src/services/sql.ts b/src/services/sql.ts index 1945efa92..671b136d2 100644 --- a/src/services/sql.ts +++ b/src/services/sql.ts @@ -144,7 +144,7 @@ function getRows(query: string, params: Params = []): T[] { } function getRawRows(query: string, params: Params = []): T[] { - return (wrap(query, s => s.raw().all(params)) as T[] | null) || []; + return (wrap(query, s => s.raw().all(params)) as T[]) || []; } function iterateRows(query: string, params: Params = []) {