mirror of
https://github.com/snapotter-hq/SnapOtter.git
synced 2026-08-03 07:46:42 +02:00
fix: address code review findings before merge
- Warn on startup if deprecated STIRLING_VARIANT env var is set - Broaden upscale.py exception handling to catch RuntimeError/OSError for Lanczos fallback (not just ImportError) - Add QEMU + multi-arch (amd64+arm64) to CI Docker build test - Use .get() instead of .all() for single-row health check query - Restore container_name in docker-compose.yml for backwards compat
This commit is contained in:
@@ -23,6 +23,14 @@ import { teamsRoutes } from "./routes/teams.js";
|
||||
import { registerToolRoutes } from "./routes/tools/index.js";
|
||||
import { userFileRoutes } from "./routes/user-files.js";
|
||||
|
||||
// Warn about deprecated STIRLING_VARIANT env var
|
||||
if (process.env.STIRLING_VARIANT) {
|
||||
console.warn(
|
||||
`WARNING: STIRLING_VARIANT="${process.env.STIRLING_VARIANT}" is set but ignored. ` +
|
||||
"There is now a single unified image with all features. Remove STIRLING_VARIANT from your environment.",
|
||||
);
|
||||
}
|
||||
|
||||
// Run before anything else
|
||||
runMigrations();
|
||||
console.log("Database initialized");
|
||||
@@ -112,7 +120,7 @@ await docsRoutes(app);
|
||||
app.get("/api/v1/health", async (_request, reply) => {
|
||||
let dbOk = false;
|
||||
try {
|
||||
db.select().from(schema.settings).limit(1).all();
|
||||
db.select().from(schema.settings).limit(1).get();
|
||||
dbOk = true;
|
||||
} catch {
|
||||
/* db unreachable */
|
||||
|
||||
Reference in New Issue
Block a user