Update healthcheck

Author: alangrainger

app/package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "notesx-api",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "scripts": {
     "dev": "ts-node src/index.ts",
     "build": "npx tsc",
@@ -9,7 +9,7 @@
   },
   "dependencies": {
     "@hono/node-server": "^1.13.8",
-    "better-sqlite3": "^11.8.1",
+    "better-sqlite3": "^12.2.0",
     "dotenv": "^16.4.5",
     "hono": "^4.7.1",
     "node-cron": "^3.0.3",

app/package.json

@@ -3,7 +3,7 @@ import { serve } from '@hono/node-server'
 import { serveStatic } from '@hono/node-server/serve-static'
 import { cors } from 'hono/cors'
 import { etag } from 'hono/etag'
-import { App, StatusCodes } from './types'
+import { App, serverError, ServerErrors, StatusCodes } from './types'
 import db from './v1/Database'
 import Cloudflare from './v1/Cloudflare'
 import log from './v1/Log'
@@ -12,6 +12,7 @@ import { router as accountRouter } from './v1/routes/account'
 import { HTTPException } from 'hono/http-exception'
 import { Cron } from './v1/Cron'
 import { trackView } from './v1/routes/middleware'
+import fs from 'fs'
 
 require('dotenv').config()
 
@@ -31,7 +32,16 @@ const app = new Hono()
 app.use('/v1/*', cors()) // CORS for all API routes
 app.route('/v1/file', fileRouter)
 app.route('/v1/account', accountRouter)
-app.get('/v1/ping', () => new Response('ok'))
+app.get('/v1/ping', async () => {
+  try {
+    // Check to make sure the upload location exists and is writeable
+    await fs.promises.access(appInstance.baseFolder, fs.constants.W_OK)
+    return new Response('ok')
+  } catch (e) {
+    console.log(e)
+    return new Response('', { status: serverError(ServerErrors.FILESYSTEM_NOT_WRITABLE) })
+  }
+})
 
 // Add etags for all files
 app.use('*', etag())
@@ -86,7 +96,7 @@ app.all('*', (c) => {
 
 app.onError((error, c) => {
   const err = error as HTTPException
-  const status = typeof err?.status === 'number' ? err.status : 500
+  const status = err.status || 500
   log.event(c, {
     status,
     endpoint: c.req.path

app/src/index.ts

@@ -1,5 +1,5 @@
-import Cloudflare from './v1/Cloudflare';
-import { Log } from './v1/Log';
+import Cloudflare from './v1/Cloudflare'
+import { Log } from './v1/Log'
 import { SQLite } from './v1/Database'
 import { ContentfulStatusCode } from 'hono/dist/types/utils/http-status'
 
@@ -25,7 +25,8 @@ export enum ServerErrors {
   FILE_FAILED_TO_UPLOAD,
   USER_FAILED_TO_SAVE,
   API_KEY_FAILED_TO_SAVE,
-  TURNSTILE_NO_VERIFY
+  TURNSTILE_NO_VERIFY,
+  FILESYSTEM_NOT_WRITABLE
 }
 
 export const StatusCodes: { [key: number]: string } = {
@@ -40,7 +41,7 @@ export const StatusCodes: { [key: number]: string } = {
   461: 'I am currently performing maintenance on the server. Service will return to normal soon.',
   462: 'Invalid API key, you should automatically be redirected to your browser to request a new one', // 462 will automatically get a new key
   463: 'Invalid authentication token'
-};
+}
 
 export function serverError (error: ServerErrors) {
   return (560 + error) as ContentfulStatusCode