From 9758ecbea217a8f2b7cbcb55571ba19d5a5d32af Mon Sep 17 00:00:00 2001 From: Daniel Onyejesi Date: Wed, 25 Mar 2026 17:53:48 -0400 Subject: [PATCH] Fix missing error handling in nextcloud disconnect, update docker-compose to v8 - Add try-catch to /nextcloud/disconnect route (was crashing on DB errors) - Update docker-compose.yml image tag from v7 to v8 - Remove unused SESSION_SECRET from .env.example --- .env.example | 1 - docker-compose.yml | 2 +- src/routes/nextcloud.js | 6 ++++-- 3 files changed, 5 insertions(+), 4 deletions(-) diff --git a/.env.example b/.env.example index 787e9c1..5129e66 100644 --- a/.env.example +++ b/.env.example @@ -47,7 +47,6 @@ ELEVENLABS_API_KEY= PORT=3000 APP_URL=https://your-domain.com JWT_SECRET=generate-a-random-64-char-string-here -SESSION_SECRET=generate-another-random-string-here # Email (for verification & password reset) SMTP_HOST=smtp.gmail.com diff --git a/docker-compose.yml b/docker-compose.yml index e069f69..d37ba9f 100644 --- a/docker-compose.yml +++ b/docker-compose.yml @@ -1,6 +1,6 @@ services: pediatric-scribe: - image: danielonyejesi/pediatric-ai-scribe-v3:v7 + image: danielonyejesi/pediatric-ai-scribe-v3:v8 ports: - "3552:3000" env_file: diff --git a/src/routes/nextcloud.js b/src/routes/nextcloud.js index 933ca8c..44c0fc1 100644 --- a/src/routes/nextcloud.js +++ b/src/routes/nextcloud.js @@ -60,8 +60,10 @@ router.post('/nextcloud/export', authMiddleware, async function(req, res) { }); router.post('/nextcloud/disconnect', authMiddleware, async function(req, res) { - await db.run('UPDATE users SET nextcloud_url = NULL, nextcloud_user = NULL, nextcloud_token = NULL, nextcloud_folder = NULL WHERE id = ?', [req.user.id]); - res.json({ success: true }); + try { + await db.run('UPDATE users SET nextcloud_url = NULL, nextcloud_user = NULL, nextcloud_token = NULL, nextcloud_folder = NULL WHERE id = ?', [req.user.id]); + res.json({ success: true }); + } catch (err) { res.status(500).json({ error: err.message }); } }); module.exports = router;