fix: tidy up ssl stuff some more

Author: soedirgo

src/lib/PostgresMetaTriggers.ts

@@ -167,9 +167,12 @@ export default class PostgresMetaTriggers {
     const enabledMode = enabled_mode.toUpperCase()
     const { name: currentName, schema: schema, table: table } = triggerRecord!
     const qualifiedTableName = `${ident(schema)}.${ident(table)}`
-    const updateNameSql = newName && newName !== currentName
-      ? `ALTER TRIGGER ${ident(currentName)} ON ${qualifiedTableName} RENAME TO ${ident(newName)};`
-      : ''
+    const updateNameSql =
+      newName && newName !== currentName
+        ? `ALTER TRIGGER ${ident(currentName)} ON ${qualifiedTableName} RENAME TO ${ident(
+            newName
+          )};`
+        : ''
 
     if (['ORIGIN', 'REPLICA', 'ALWAYS', 'DISABLED'].includes(enabledMode)) {
       if (enabledMode === 'DISABLED') {

src/lib/db.ts

@@ -3,18 +3,16 @@ import { PostgresMetaResult } from './types'
 
 types.setTypeParser(20, parseInt)
 
-export const init: (
-  config: PoolConfig
-) => {
+export const init: (config: PoolConfig) => {
   query: (sql: string) => Promise<PostgresMetaResult<any>>
   end: () => Promise<void>
 } = (config) => {
-  // XXX: Race condition could happen here: one async task may be doing
+  // NOTE: Race condition could happen here: one async task may be doing
   // `pool.end()` which invalidates the pool and subsequently all existing
   // handles to `query`. Normally you might only deal with one DB so you don't
   // need to call `pool.end()`, but since the server needs this, we make a
   // compromise: if we run `query` after `pool.end()` is called (i.e. pool is
-  // `null`), we temporarily create a pool and close is right after.
+  // `null`), we temporarily create a pool and close it right after.
   let pool: Pool | null = new Pool(config)
   return {
     async query(sql) {

src/server/constants.ts

@@ -7,11 +7,8 @@ const PG_META_DB_USER = process.env.PG_META_DB_USER || 'postgres'
 const PG_META_DB_PORT = Number(process.env.PG_META_DB_PORT) || 5432
 const PG_META_DB_PASSWORD = process.env.PG_META_DB_PASSWORD || 'postgres'
 
-export const PG_CONNECTION = `postgres://${PG_META_DB_USER}:${PG_META_DB_PASSWORD}@${PG_META_DB_HOST}:${PG_META_DB_PORT}/${PG_META_DB_NAME}`
+export const PG_CONNECTION = `postgres://${PG_META_DB_USER}:${PG_META_DB_PASSWORD}@${PG_META_DB_HOST}:${PG_META_DB_PORT}/${PG_META_DB_NAME}?sslmode=disable`
 
 export const PG_META_EXPORT_DOCS = process.env.PG_META_EXPORT_DOCS === 'true' || false
 
-export const DEFAULT_POOL_CONFIG =
-  process.env.NODE_ENV === 'development'
-    ? { max: 1, ssl: false }
-    : { max: 1, ssl: { rejectUnauthorized: false } }
+export const DEFAULT_POOL_CONFIG = { max: 1 }