c2cwsgiutils 6.2.0.dev154__tar.gz → 6.2.0.dev161__tar.gz

{c2cwsgiutils-6.2.0.dev154 → c2cwsgiutils-6.2.0.dev161}/PKG-INFO

@@ -1,8 +1,9 @@
-Metadata-Version: 2.3
+Metadata-Version: 2.4
 Name: c2cwsgiutils
-Version: 6.2.0.dev154
+Version: 6.2.0.dev161
 Summary: Common utilities for Camptocamp WSGI applications
-License: BSD-2-Clause
+License-Expression: BSD-2-Clause
+License-File: LICENSE
 Keywords: geo,gis,sqlalchemy,orm,wsgi
 Author: Camptocamp
 Author-email: info@camptocamp.com

{c2cwsgiutils-6.2.0.dev154 → c2cwsgiutils-6.2.0.dev161}/c2cwsgiutils/db.py

@@ -51,7 +51,7 @@ def setup_session(
     With an accompanying tween that switches between the master and the slave DB
     connection. Uses prefixed entries in the application's settings.
 
-    The slave DB will be used for anything that is GET and OPTIONS queries. The master DB will be used for
+    The slave DB will be used for anything that is GET, HEAD and OPTIONS queries. The master DB will be used for
     all the other queries. You can tweak this behavior with the force_master and force_slave parameters.
     Those parameters are lists of regex that are going to be matched against "{VERB} {PATH}". Warning, the
     path includes the route_prefix.
@@ -111,7 +111,7 @@ def create_session(
     With an accompanying tween that switches between the master and the slave DB
     connection.
 
-    The slave DB will be used for anything that is GET and OPTIONS queries. The master DB will be used for
+    The slave DB will be used for anything that is GET, HEAD and OPTIONS queries. The master DB will be used for
     all the other queries. You can tweak this behavior with the force_master and force_slave parameters.
     Those parameters are lists of regex that are going to be matched against "{VERB} {PATH}". Warning, the
     path includes the route_prefix.
@@ -187,7 +187,10 @@ def _add_tween(
             has_force_master = any(r.match(method_path) for r in master_paths)
             if FORCE_READONLY or (
                 not has_force_master
-                and (request.method in ("GET", "OPTIONS") or any(r.match(method_path) for r in slave_paths))
+                and (
+                    request.method in ("GET", "HEAD", "OPTIONS")
+                    or any(r.match(method_path) for r in slave_paths)
+                )
             ):
                 _LOG.debug(
                     "Using %s database for: %s",
@@ -260,7 +263,7 @@ class SessionFactory(_sessionmaker):
             if FORCE_READONLY or (
                 not has_force_master
                 and (
-                    request.method in ("GET", "OPTIONS")
+                    request.method in ("GET", "OPTIONS", "HEAD")
                     or any(r.match(method_path) for r in self.slave_paths)
                 )
             ):

{c2cwsgiutils-6.2.0.dev154 → c2cwsgiutils-6.2.0.dev161}/c2cwsgiutils/stats_pyramid/_db_spy.py

@@ -16,7 +16,7 @@ _LOG = logging.getLogger(__name__)
 _PROMETHEUS_DB_SUMMARY = prometheus_client.Summary(
     prometheus.build_metric_name("database"),
     "Database requests",
-    ["what"],
+    ["what", "engine"],
     unit="seconds",
 )
 
@@ -67,12 +67,14 @@ def _simplify_sql(sql: str) -> str:
     return re.sub(r"%\(\w+\)\w", "?", sql)
 
 
-def _create_sqlalchemy_timer_cb(what: str) -> Callable[..., Any]:
+def _create_sqlalchemy_timer_cb(what: str, engine_name: str | None = None) -> Callable[..., Any]:
     start = time.perf_counter()
 
     def after(*_args: Any, **_kwargs: Any) -> None:
-        _PROMETHEUS_DB_SUMMARY.labels({"query": what}).observe(time.perf_counter() - start)
-        _LOG.debug("Execute statement '%s' in %d.", what, time.perf_counter() - start)
+        elapsed_time = time.perf_counter() - start
+        _PROMETHEUS_DB_SUMMARY.labels(what=what, engine=engine_name).observe(elapsed_time)
+        engine_suffix = f", on {engine_name}" if engine_name else ""
+        _LOG.debug("Execute statement '%s' in %.3f%ss.", what, elapsed_time, engine_suffix)
 
     return after
 
@@ -85,10 +87,11 @@ def _before_cursor_execute(
     _context: Any,
     _executemany: Any,
 ) -> None:
+    engine_name = getattr(conn.engine, "c2c_name", None)
     sqlalchemy.event.listen(
         conn,
         "after_cursor_execute",
-        _create_sqlalchemy_timer_cb(_simplify_sql(statement)),
+        _create_sqlalchemy_timer_cb(_simplify_sql(statement), engine_name),
         once=True,
     )
 

{c2cwsgiutils-6.2.0.dev154 → c2cwsgiutils-6.2.0.dev161}/pyproject.toml

@@ -227,7 +227,7 @@ include = [
 ]
 requires-python = ">=3.10"
 dependencies = ["requests", "pyyaml", "ujson", "cee_syslog_handler", "prometheus-client"]
-version = "6.2.0.dev154"
+version = "6.2.0.dev161"
 
 [project.urls]
 repository = "https://github.com/camptocamp/c2cwsgiutils"