pgbelt 0.7.3__tar.gz → 0.7.5__tar.gz

{pgbelt-0.7.3 → pgbelt-0.7.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: pgbelt
-Version: 0.7.3
+Version: 0.7.5
 Summary: A CLI tool used to manage Postgres data migrations from beginning to end, for a single database or a fleet, leveraging pglogical replication.
 Author: Varjitt Jeeva
 Author-email: varjitt.jeeva@autodesk.com
@@ -10,7 +10,7 @@ Classifier: Programming Language :: Python :: 3.9
 Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
-Requires-Dist: aiofiles (>=0.8,<23.3)
+Requires-Dist: aiofiles (>=0.8,<24.2)
 Requires-Dist: asyncpg (>=0.27,<0.30)
 Requires-Dist: pydantic (>=2.0,<3.0)
 Requires-Dist: tabulate (>=0.9.0,<0.10.0)

{pgbelt-0.7.3 → pgbelt-0.7.5}/pgbelt/cmd/login.py

@@ -55,7 +55,12 @@ async def revoke_logins(config_future: Awaitable[DbupgradeConfig]) -> None:
             await _populate_logins(conf.src, pool, logger)
             save_task = asyncio.create_task(conf.save())
 
-        to_disable = [conf.src.owner_user.name]
+        to_disable = []
+        # Sometimes the owner user is the same as the root user.
+        # When that happens, we don't want to disable the owner user.
+        # If the owner user is different, we want to disable the owner user.
+        if conf.src.owner_user.name != conf.src.root_user.name:
+            to_disable.append(conf.src.owner_user.name)
 
         if conf.src.other_users is not None:
             to_disable += [

{pgbelt-0.7.3 → pgbelt-0.7.5}/pgbelt/cmd/teardown.py

@@ -42,13 +42,13 @@ async def teardown_forward_replication(config_future: Awaitable[DbupgradeConfig]
 @run_with_configs
 async def teardown(
     config_future: Awaitable[DbupgradeConfig],
-    full: bool = Option(False, help="Remove pglogical user and extension"),
+    full: bool = Option(False, help="Remove pglogical extension"),
 ):
     """
     Removes all pglogical configuration from both databases. If any replication is
-    configured this will stop it.
+    configured this will stop it. It will also drop the pglogical user.
 
-    If run with --full the pglogical users and extension will be dropped.
+    If run with --full the pglogical extension will be dropped.
 
     WARNING: running with --full may cause the database to lock up. You should be
     prepared to reboot the database if you do this.

{pgbelt-0.7.3 → pgbelt-0.7.5}/pgbelt/config/models.py

@@ -10,6 +10,7 @@ from pgbelt.util.asyncfuncs import makedirs
 from pydantic import BaseModel
 from pydantic import ValidationError
 from pydantic import field_validator
+from urllib.parse import quote
 
 
 def config_dir(db: str, dc: str) -> str:
@@ -86,15 +87,24 @@ class DbConfig(BaseModel):
 
     @property
     def root_uri(self) -> str:
-        return f"postgresql://{self.root_user.name}:{self.root_user.pw}@{self.ip}:{self.port}/{self.db}"
+        password = quote(
+            self.root_user.pw
+        )  # https://github.com/encode/databases/issues/145#issuecomment-1303792343 need this to handle special characters
+        return f"postgresql://{self.root_user.name}:{password}@{self.ip}:{self.port}/{self.db}"
 
     @property
     def owner_uri(self) -> str:
-        return f"postgresql://{self.owner_user.name}:{self.owner_user.pw}@{self.ip}:{self.port}/{self.db}"
+        password = quote(
+            self.owner_user.pw
+        )  # https://github.com/encode/databases/issues/145#issuecomment-1303792343 need this to handle special characters
+        return f"postgresql://{self.owner_user.name}:{password}@{self.ip}:{self.port}/{self.db}"
 
     @property
     def pglogical_uri(self) -> str:
-        return f"postgresql://{self.pglogical_user.name}:{self.pglogical_user.pw}@{self.ip}:{self.port}/{self.db}"
+        password = quote(
+            self.pglogical_user.pw
+        )  # https://github.com/encode/databases/issues/145#issuecomment-1303792343 need this to handle special characters
+        return f"postgresql://{self.pglogical_user.name}:{password}@{self.ip}:{self.port}/{self.db}"
 
 
 class DbupgradeConfig(BaseModel):

{pgbelt-0.7.3 → pgbelt-0.7.5}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "pgbelt"
-version = "0.7.3"
+version = "0.7.5"
 description = "A CLI tool used to manage Postgres data migrations from beginning to end, for a single database or a fleet, leveraging pglogical replication."
 authors = ["Varjitt Jeeva <varjitt.jeeva@autodesk.com>"]
 readme = "README.md"
@@ -11,34 +11,34 @@ packages = [
 
 [tool.poetry.dependencies]
 python = ">=3.9,<4.0"
-aiofiles = ">=0.8,<23.3"
+aiofiles = ">=0.8,<24.2"
 asyncpg = ">=0.27,<0.30"
 pydantic = ">=2.0,<3.0"
 tabulate = "^0.9.0"
 typer = ">=0.9,<0.13"
 
 [tool.poetry.dev-dependencies]
-black = "~24.3.0"
-pre-commit = "~3.7.0"
-flake8 = "^7.0.0"
+black = "~24.4.2"
+pre-commit = "~3.7.1"
+flake8 = "^7.1.0"
 pytest-cov = "~5.0.0"
-pytest = "^8.1.1"
-coverage = {extras = ["toml"], version = "^7.4"}
-safety = "^3.1.0"
-mypy = "^1.9"
-xdoctest = {extras = ["colors"], version = "^1.1.3"}
+pytest = "^8.2.2"
+coverage = {extras = ["toml"], version = "^7.5"}
+safety = "^3.2.4"
+mypy = "^1.10"
+xdoctest = {extras = ["colors"], version = "^1.1.5"}
 flake8-bandit = "~4.1.1"
 flake8-bugbear = ">=21.9.2"
 flake8-docstrings = "^1.6.0"
 flake8-rst-docstrings = "^0.3.0"
-pep8-naming = "^0.13.2"
+pep8-naming = "^0.14.1"
 darglint = "^1.8.1"
-reorder-python-imports = "^3.9.0"
-pre-commit-hooks = "^4.5.0"
-Pygments = "^2.17.2"
-pyupgrade = "^3.15.2"
-pylint = "^3.1.0"
-pytest-asyncio = "~0.23.6"
+reorder-python-imports = "^3.13.0"
+pre-commit-hooks = "^4.6.0"
+Pygments = "^2.18.0"
+pyupgrade = "^3.16.0"
+pylint = "^3.2.5"
+pytest-asyncio = "~0.23.7"
 
 [build-system]
 requires = ["poetry-core>=1.0.0", "setuptools"]