Merge pull request #5 from mlibrary/fix-sqlalchemy-connection-url

Fix: Alembic needs a connection string
mlibrary · Sep 25, 2024 · 59e35a5 · 59e35a5
2 parents 2cb8c17 + 85a3c57
commit 59e35a5
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/aim/digifeeds/database/migrations/env.py b/aim/digifeeds/database/migrations/env.py
@@ -13,7 +13,11 @@
 config = context.config
 
 # Set url from environment variable
-config.set_main_option("sqlalchemy.url", S.mysql_database)
+config.set_main_option(
+    "sqlalchemy.url",
+    # This is for handling passwords with special characters
+    S.mysql_database.render_as_string(hide_password=False).replace("%", "%%"),
+)
 
 # Interpret the config file for Python logging.
 # This line sets up loggers basically.

diff --git a/compose.yml b/compose.yml
@@ -28,7 +28,7 @@ services:
     environment:
       - MARIADB_ROOT_PASSWORD=password
       - MARIADB_USER=user
-      - MARIADB_PASSWORD=password
+      - MARIADB_PASSWORD=test@123
       - MARIADB_DATABASE=database
 
 volumes:

diff --git a/env.development b/env.development
@@ -1,5 +1,5 @@
 MARIADB_ROOT_PASSWORD=password
 MARIADB_USER=user
-MARIADB_PASSWORD=password
+MARIADB_PASSWORD=test@123
 MARIADB_DATABASE=database
 DATABASE_HOST=database