From ecdae702ad021307ca94eec3ec8cdde3123924bf Mon Sep 17 00:00:00 2001
From: funkecoder23 <12570656+FunkeCoder23@users.noreply.github.com>
Date: Sun, 4 Feb 2024 23:23:57 -0500
Subject: [PATCH] Remove alter param from database sync

Re: [sequelize docs](https://sequelize.org/docs/v6/core-concepts/model-basics/#synchronization-in-production)
Alter is a bad idea in production.

Also print error message instead of sequelize error object as it's a bit verbose
---
 src/node/consumer/src/lib/repository.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/node/consumer/src/lib/repository.js b/src/node/consumer/src/lib/repository.js
index 96a75a4..a71793f 100644
--- a/src/node/consumer/src/lib/repository.js
+++ b/src/node/consumer/src/lib/repository.js
@@ -1,7 +1,7 @@
 import moment from 'moment';
 import { Sequelize, Op, DataTypes, fn, col, literal } from 'sequelize';
 import { databaseConfig } from './config.js';
-import {logger} from "./logger.js";
+import { logger } from "./logger.js";
 import * as Promises from './promises.js';
 
 const database = new Sequelize(
@@ -185,9 +185,9 @@ Subtitle.belongsTo(File, { foreignKey: 'fileId', constraints: false });
 
 export function connect() {
     if (databaseConfig.ENABLE_SYNC) {
-        return database.sync({ alter: true })
+        return database.sync()
             .catch(error => {
-                console.error('Failed syncing database: ', error);
+                console.error('Failed syncing database: ', error.message);
                 throw error;
             });
     }