Merge pull request #170 from brooklyn-data/escape_quotes

Escape quotes
brooklyn-data · Aug 12, 2022 · 63c4c8f · 63c4c8f
2 parents 3ab5099 + 5dcd914
commit 63c4c8f
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/integration_test_project/models/sources.yml b/integration_test_project/models/sources.yml
@@ -7,6 +7,6 @@ sources:
     freshness:
       error_after: {count: 24, period: hour}
       filter: dayname(updatedat) not in ('Sunday', 'Monday')
-    loaded_at_field: current_timestamp
+    loaded_at_field: convert_timezone('UTC', load_timestamp)
     tables:
       - name: doesnt_exist
diff --git a/integration_test_project/models/tests_and_exposures.yml b/integration_test_project/models/tests_and_exposures.yml
@@ -13,7 +13,7 @@ exposures:
   - name: imaginary
     type: dashboard
     maturity: high
-    description: "dashboard description"
+    description: "ceo's favourite dashboard"
     url: https://bi.tool/dashboards/1
 
     depends_on:

diff --git a/macros/upload_exposures.sql b/macros/upload_exposures.sql
@@ -31,7 +31,7 @@
                 '{{ tojson(exposure.owner) }}', {# owner #}
                 '{{ exposure.maturity }}', {# maturity #}
                 '{{ exposure.original_file_path | replace('\\', '\\\\') }}', {# path #}
-                '{{ exposure.description }}', {# description #}
+                '{{ exposure.description | replace("'","\\'") }}', {# description #}
                 '{{ exposure.url }}', {# url #}
                 '{{ exposure.package_name }}', {# package_name #}
                 '{{ tojson(exposure.depends_on.nodes) }}' {# depends_on_nodes #}

diff --git a/macros/upload_sources.sql b/macros/upload_sources.sql
@@ -31,7 +31,7 @@
                 '{{ source.loader }}', {# loader #}
                 '{{ source.name }}', {# name #}
                 '{{ source.identifier }}', {# identifier #}
-                '{{ source.loaded_at_field }}', {# loaded_at_field #}
+                '{{ source.loaded_at_field | replace("'","\\'") }}', {# loaded_at_field #}
                 '{{ tojson(source.freshness) | replace("'","\\'") }}' {# freshness #}
             )
             {%- if not loop.last %},{%- endif %}