Centralize hashing

packages/sshecret-admin/src/sshecret_admin/core/cli.py

@@ -5,13 +5,13 @@ import code
 from collections.abc import Awaitable
 import logging
 from typing import Any, cast
-import bcrypt
 import click
 from sshecret_admin.services.admin_backend import AdminBackend
 import uvicorn
 from pydantic import ValidationError
 from sqlmodel import Session, create_engine, select
 from sshecret_admin.auth.models import init_db, User, PasswordDB
+from sshecret_admin.auth.authentication import hash_password
 from sshecret_admin.core.settings import AdminServerSettings
 
 handler = logging.StreamHandler()
@@ -19,17 +19,6 @@ formatter = logging.Formatter(
     "%(asctime)s [%(processName)s: %(process)d] [%(threadName)s: %(thread)d] [%(levelname)s] %(name)s: %(message)s"
 )
 handler.setFormatter(formatter)
-LOG = logging.getLogger()
-LOG.addHandler(handler)
-LOG.setLevel(logging.INFO)
-
-
-def hash_password(password: str) -> str:
-    """Hash password."""
-    salt = bcrypt.gensalt()
-    hashed_password = bcrypt.hashpw(password.encode(), salt)
-    return hashed_password.decode()
-
 
 def create_user(session: Session, username: str, password: str) -> None:
     """Create a user."""
@@ -44,8 +33,14 @@ def create_user(session: Session, username: str, password: str) -> None:
 @click.pass_context
 def cli(ctx: click.Context, debug: bool) -> None:
     """Sshecret Admin."""
+    LOG = logging.getLogger()
+    LOG.addHandler(handler)
+
     if debug:
+        click.echo("Setting logging to debug level")
         LOG.setLevel(logging.DEBUG)
+    else:
+        LOG.setLevel(logging.INFO)
     try:
         settings = AdminServerSettings()  # pyright: ignore[reportCallIssue]
     except ValidationError as e:

packages/sshecret-admin/src/sshecret_admin/core/db.py

@@ -12,7 +12,7 @@ def setup_database(
 ) -> tuple[sa.Engine, Callable[[], Generator[Session, None, None]]]:
     """Setup database."""
 
-    engine = create_engine(db_url, echo=True)
+    engine = create_engine(db_url, echo=False)
 
     def get_db_session() -> Generator[Session, None, None]:
         """Get DB Session."""