Merge pull request #50 from SEKOIA-IO/feat/sync_library_env_variable

feat: Allows to use env variables for secrets in sync library script

Author: Darkheir

CHANGELOG.md

@@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Changed
 
 - In `synchronize-lib` script add support for slug based docker names
+- In `synchronize-lib` script allow to use environment variable for secrets
 
 ## [1.3.0] - 2023-06-06
 

sekoia_automation/cli.py

@@ -139,15 +139,23 @@ def openapi_to_module(
 
 @app.command(name="synchronize-library")
 def sync_library(
-    playbook_url=typer.Argument(..., help="URL of the Playbook API"),
-    api_key=typer.Argument(..., help="Secret key to connect to the Playbook API"),
-    modules_path: Path = typer.Argument(".", help="Path to the playbook modules"),
+    playbook_url=typer.Argument(
+        ..., envvar="PLAYBOOK_URL", help="URL of the Playbook API"
+    ),
+    api_key=typer.Argument(
+        ..., envvar="PLAYBOOK_API_KEY", help="Secret key to connect to the Playbook API"
+    ),
+    modules_path: Path = typer.Option(".", help="Path to the playbook modules"),
     module: str = typer.Option("", help="Module to deploy. Default to all modules"),
     check_image_on_registry: bool = typer.Option(
         False, help="Whether to check registry for existing image"
     ),
-    registry_pat: str = typer.Option("", help="Docker registry personal access token"),
-    registry_user: str = typer.Option("", help="Docker registry username"),
+    registry_pat: str = typer.Option(
+        "", envvar="REGISTRY_PAT", help="Docker registry personal access token"
+    ),
+    registry_user: str = typer.Option(
+        "", envvar="REGISTRY_USER", help="Docker registry username"
+    ),
 ):
     """
     Synchronize the module library to Sekoia.io

tests/test_cli.py

@@ -199,3 +199,11 @@ def test_openapi_url_to_module(tmp_path, requests_mock, swagger_path):
     assert module.joinpath("main.py").exists()
     assert len(list(module.glob("action_*"))) > 0
     assert module.joinpath("dashboard_api", "__init__.py").exists()
+
+
+def test_synchronize_library():
+    res: Result = runner.invoke(
+        app,
+        ["synchronize-library", "foo", "bar"],
+    )
+    assert res.exit_code == 0