Where should db.close() be called?

[dpointeck]

Hi there,

I'm new to Deno and use Denodb with Sqlite for a small side project - https://github.com/dpointeck/dag.
I really like Denodb it has a nice API. I started out with this server.ts.

import { Application } from "https://deno.land/x/oak/mod.ts";
import { api } from "./api.ts";
import { db } from "./data/Database.ts";

const port = 8080;
const app = new Application();

// Init DB
await db.sync();

// Middleware
app.use(logger);

// Routes
app.use(api.routes());

// Close db
db.close()

app.addEventListener("listen", () => {
  console.log(`⚡️⚡️⚡️ Listening on localhost:${port}`);
});

await app.listen({ port });

From this I always got a server error. But when I leave db.close() away the the app is running without problems.

import { Application } from "https://deno.land/x/oak/mod.ts";
import { api } from "./api.ts";
import { db } from "./data/Database.ts";

const port = 8080;
const app = new Application();

// Init DB
await db.sync();

// Middleware
app.use(logger);

// Routes
app.use(api.routes());

app.addEventListener("listen", () => {
  console.log(`⚡️⚡️⚡️ Listening on localhost:${port}`);
});

await app.listen({ port });

I also saw a tutorial where the db.sync() call was moved below the routes like in the snippet below. Also this works fine.

import { Application } from "https://deno.land/x/oak/mod.ts";
import { api } from "./api.ts";
import { db } from "./data/Database.ts";

const port = 8080;
const app = new Application();

// Middleware
app.use(logger);

// Routes
app.use(api.routes());

// Init DB
await db.sync();

app.addEventListener("listen", () => {
  console.log(`⚡️⚡️⚡️ Listening on localhost:${port}`);
});

await app.listen({ port });

At the moment I'm not able to figure out what's the right way and I'm also strugging with this call to db.sync(). I really want to know it is necessary and if yes where to put it and why.