api-foundry-query-engine 0.8.77__tar.gz → 0.8.81__tar.gz

{api_foundry_query_engine-0.8.77 → api_foundry_query_engine-0.8.81}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: api-foundry-query-engine
-Version: 0.8.77
+Version: 0.8.81
 Summary: The AWS lambda service handler use by the `api_foundry` project is a powerful tool designed to automate the deployment of REST APIs on AWS using Lambda services to access and interact with relational databases (RDBMS). This project leverages the OpenAPI specification to define and manage the APIs
 Project-URL: Documentation, https://github.com/DanRepik/api-foundry
 Project-URL: Source, https://github.com/DanRepik/api-foundry

api_foundry_query_engine-0.8.81/api_foundry_query_engine/__init__.py

@@ -0,0 +1 @@
+__version__ = "0.8.81"

{api_foundry_query_engine-0.8.77 → api_foundry_query_engine-0.8.81}/api_foundry_query_engine/connectors/connection_factory.py

@@ -59,9 +59,7 @@ class ConnectionFactory:
 
         engine = db_config.get("engine")
         if not engine:
-            raise ApplicationException(
-                500, "Database 'engine' is not defined in the secret."
-            )
+            raise ApplicationException(500, "Database 'engine' is not defined in the secret.")
 
         if engine == "postgres":
             from .postgres_connection import PostgresConnection

{api_foundry_query_engine-0.8.77 → api_foundry_query_engine-0.8.81}/api_foundry_query_engine/dao/sql_query_handler.py

@@ -382,6 +382,15 @@ class SQLQueryHandler:
         return sql
 
     def generate_placeholders(self, property: SchemaObjectProperty, value, prefix: Optional[str] = None) -> dict:
+        placeholder_name = f"{prefix}_{property.api_name}" if prefix else property.api_name
+        if value is None:
+            # An unset optional query parameter must bind an actual SQL
+            # NULL, not the string "None" -- str(None) below would
+            # otherwise produce a literal 'None' value, silently
+            # breaking every `:param IS NULL OR ...`-style optional
+            # filter (e.g. date-range parameters on a custom SQL route).
+            return {placeholder_name: None}
+
         operand = "="
 
         if isinstance(value, str):
@@ -393,7 +402,6 @@ class SQLQueryHandler:
         else:
             value_str = str(value)
 
-        placeholder_name = f"{prefix}_{property.api_name}" if prefix else property.api_name
         placeholders = {}
 
         if operand in ["between", "not-between"]:

{api_foundry_query_engine-0.8.77 → api_foundry_query_engine-0.8.81}/api_foundry_query_engine/dao/sql_select_query_handler.py

@@ -281,10 +281,21 @@ class SQLSelectSchemaQueryHandler(SQLSchemaQueryHandler):
                     schema_object.permissions,
                     schema_object.properties,
                 )
+                # Only genuine joined relations need their columns
+                # qualified (e.g. "customer.name", to disambiguate from
+                # the parent's own "name") -- the primary entity's own
+                # properties come back from the cursor under their bare
+                # column names, and marshal_record/the post-fetch column
+                # filter in PostgresCursor.execute both key off that.
+                # Prefixing them too (self.prefix_map[relation], as
+                # before) produced selection_results keys like "q.slug"
+                # that never matched any real result column, so every
+                # row came back empty and marshal_record KeyError'd
+                # trying to unwrap it.
                 filtered_keys = self.filter_and_prefix_keys(
                     reg_exs,
                     allowed_properties,
-                    self.prefix_map[relation],
+                    self.prefix_map[relation] if relation_property else None,
                 )
 
                 # Extend the result map with the filtered keys

api_foundry_query_engine-0.8.77/api_foundry_query_engine/__init__.py

@@ -1 +0,0 @@
-__version__ = "0.8.77"