Merge pull request #48 from MikeAmputer/kravtsov/parameters-in-body

Send parameters in body
MikeAmputer · Jul 19, 2024 · a32970c · a32970c
2 parents 4eb94b3 + 4a5444e
commit a32970c
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 1 deletion.
diff --git a/src/ClickHouse.Facades/ClickHouse.Facades.csproj b/src/ClickHouse.Facades/ClickHouse.Facades.csproj
@@ -33,7 +33,7 @@
 	</PropertyGroup>
 
 	<ItemGroup>
-		<PackageReference Include="ClickHouse.Client" Version="7.5.3" />
+		<PackageReference Include="ClickHouse.Client" Version="7.6.0" />
 		<PackageReference Include="System.Linq.Async" Version="6.0.1" />
 	</ItemGroup>
 

diff --git a/src/ClickHouse.Facades/Context/Context/ClickHouseContext.cs b/src/ClickHouse.Facades/Context/Context/ClickHouseContext.cs
@@ -115,6 +115,8 @@ internal async Task Initialize(ClickHouseContextOptions<TContext> options)
 
 		_connection = CreateConnection(options);
 
+		_connection.SetFormDataParameters(options.ParametersInBody);
+
 		_connectionBroker = options.ConnectionBrokerProvider(
 			_connection,
 			ICommandExecutionStrategy.Pick(options.CommandExecutionStrategy));

diff --git a/src/ClickHouse.Facades/Context/Context/ClickHouseContextOptions.cs b/src/ClickHouse.Facades/Context/Context/ClickHouseContextOptions.cs
@@ -28,4 +28,6 @@ internal Func<ClickHouseConnection, ICommandExecutionStrategy, IClickHouseConnec
 	internal CommandExecutionStrategy CommandExecutionStrategy { get; set; } = CommandExecutionStrategy.Default;
 
 	internal TransactionBrokerOptions TransactionBrokerOptions { get; set; } = null!;
+
+	internal bool ParametersInBody { get; set; } = false;
 }
diff --git a/src/ClickHouse.Facades/Context/Context/ClickHouseContextOptionsBuilder.cs b/src/ClickHouse.Facades/Context/Context/ClickHouseContextOptionsBuilder.cs
@@ -26,6 +26,8 @@ private OptionalValue<
 
 	private OptionalValue<Action<TransactionBrokerOptionsBuilder>> _setupTransactionBrokerOptions;
 
+	private OptionalValue<bool> _parametersInBody;
+
 	public ClickHouseContextOptionsBuilder<TContext> SetupTransactions(
 		Action<TransactionBrokerOptionsBuilder> setup)
 	{
@@ -135,6 +137,14 @@ internal ClickHouseContextOptionsBuilder<TContext> WithConnectionBrokerProvider(
 			connectionBrokerProvider);
 	}
 
+	public ClickHouseContextOptionsBuilder<TContext> SendParametersInBody()
+	{
+		return WithPropertyValue(
+			builder => builder._parametersInBody,
+			(builder, value) => builder._parametersInBody = value,
+			true);
+	}
+
 	protected override ClickHouseContextOptions<TContext> BuildCore()
 	{
 		var connectionString = _connectionString.NotNullOrThrow();
@@ -160,6 +170,7 @@ protected override ClickHouseContextOptions<TContext> BuildCore()
 			ConnectionBrokerProvider = _connectionBrokerProvider.NotNullOrThrow(),
 			CommandExecutionStrategy = _commandExecutionStrategy.OrElseValue(CommandExecutionStrategy.Default),
 			TransactionBrokerOptions = transactionBrokerOptions,
+			ParametersInBody = _parametersInBody.OrDefault(),
 		};
 	}