fix: prevent breaking changes

Author: ryands17

src/config.rs

@@ -221,8 +221,7 @@ pub struct Database {
 
     // sqlite configuration
     /// set the foreign key pragma to be enabled or disabled
-    #[serde(default)]
-    pub enable_foreign_keys: bool,
+    pub enable_foreign_keys: Option<bool>,
 }
 
 #[derive(Debug, Clone, Deserialize, Serialize)]

src/db.rs

@@ -150,11 +150,11 @@ pub async fn connect(config: &config::Database) -> Result<DbConn, sea_orm::DbErr
     let db = Database::connect(opt).await?;
 
     if db.get_database_backend() == DatabaseBackend::Sqlite {
-        let foreign_keys = if config.enable_foreign_keys {
-            "ON"
-        } else {
-            "OFF"
-        };
+        // default set to "ON" to prevent breaking changes
+        let foreign_keys = config
+            .enable_foreign_keys
+            .map(|fk| if fk { "ON" } else { "OFF" })
+            .unwrap_or("ON");
 
         db.execute(Statement::from_string(
             DatabaseBackend::Sqlite,

src/tests_cfg/config.rs

@@ -36,7 +36,7 @@ pub fn test_config() -> Config {
             auto_migrate: false,
             dangerously_truncate: false,
             dangerously_recreate: false,
-            enable_foreign_keys: false,
+            enable_foreign_keys: None,
         },
         queue: None,
         auth: None,