fix query string escaping

lib/ch/query.ex

@@ -123,7 +123,15 @@ defimpl DBConnection.Query, for: Ch.Query do
 
   defp encode_param(n) when is_integer(n), do: Integer.to_string(n)
   defp encode_param(f) when is_float(f), do: Float.to_string(f)
-  defp encode_param(b) when is_binary(b), do: escape_param([{"\t", "\\t"}, {"\n", "\\n"}], b)
+
+  # TODO possibly speed up
+  # For more info see
+  # https://clickhouse.com/docs/en/interfaces/http#tabs-in-url-parameters
+  # "escaped" format is the same as https://clickhouse.com/docs/en/interfaces/formats#tabseparated-data-formatting
+  defp encode_param(b) when is_binary(b) do
+    escape_param([{"\\", "\\\\"}, {"\t", "\\\t"}, {"\n", "\\\n"}], b)
+  end
+
   defp encode_param(b) when is_boolean(b), do: Atom.to_string(b)
   defp encode_param(%Decimal{} = d), do: Decimal.to_string(d, :normal)
   defp encode_param(%Date{} = date), do: Date.to_iso8601(date)

test/ch/query_string_test.exs

@@ -5,7 +5,19 @@ defmodule Ch.QueryStringTest do
     {:ok, conn: start_supervised!({Ch, database: Ch.Test.database()})}
   end
 
+  # For more info see
+  # https://clickhouse.com/docs/en/interfaces/http#tabs-in-url-parameters
+  # "escaped" format is the same as https://clickhouse.com/docs/en/interfaces/formats#tabseparated-data-formatting
   test "binaries are escaped properly", %{conn: conn} do
-    assert Ch.query!(conn, "select {s:String}", %{"s" => "\\"}).rows == []
+    for s <- ["\t", "\n", "\\", "'", "\b", "\f", "\r", "\0"] do
+      assert Ch.query!(conn, "select {s:String}", %{"s" => s}).rows == [[s]]
+    end
+
+    # example from https://clickhouse.com/docs/en/interfaces/http#tabs-in-url-parameters
+    assert Ch.query!(conn, "select splitByChar('\t', 'abc\t123')").rows ==
+             [[["abc", "123"]]]
+
+    assert Ch.query!(conn, "select splitByChar('\t', {arg1:String})", %{"arg1" => "abc\t123"}).rows ==
+             [[["abc", "123"]]]
   end
 end