Skip to content

Commit 08d5280

Browse files
authored
Remove http proxying feature flag (#9341)
1 parent 4bd345d commit 08d5280

File tree

3 files changed

+4
-6
lines changed

3 files changed

+4
-6
lines changed

release_notes.md

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,5 +6,4 @@
66

77
- Patch Durable Functions Extension version check (https://github.com/Azure/azure-functions-host/pull/9331)
88
- Update Java Worker Version to [2.12.0](https://github.com/Azure/azure-functions-java-worker/releases/tag/2.12.0)
9-
10-
9+
- Remove feature flag for http proxying (https://github.com/Azure/azure-functions-host/pull/9341)

src/WebJobs.Script.Grpc/Channel/GrpcWorkerChannel.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -430,7 +430,7 @@ internal void WorkerInitResponse(GrpcEvent initEvent)
430430

431431
// If http proxying is enabled, we need to get the proxying endpoint of this worker
432432
var httpUri = _workerCapabilities.GetCapabilityState(RpcWorkerConstants.HttpUri);
433-
if (!string.IsNullOrEmpty(httpUri) && FeatureFlags.IsEnabled(ScriptConstants.FeatureFlagEnableHttpProxying))
433+
if (!string.IsNullOrEmpty(httpUri))
434434
{
435435
try
436436
{
@@ -763,7 +763,7 @@ await SendStreamingMessageAsync(new StreamingMessage
763763
context.CancellationToken.Register(() => SendInvocationCancel(invocationRequest.InvocationId));
764764
}
765765

766-
if (IsHttpProxyingWorker && FeatureFlags.IsEnabled(ScriptConstants.FeatureFlagEnableHttpProxying) && context.FunctionMetadata.IsHttpTriggerFunction())
766+
if (IsHttpProxyingWorker && context.FunctionMetadata.IsHttpTriggerFunction())
767767
{
768768
var aspNetTask = _httpProxyService.ForwardAsync(context, _httpProxyEndpoint).AsTask();
769769

@@ -963,7 +963,7 @@ internal async Task InvokeResponse(InvocationResponse invokeResponse)
963963
{
964964
if (invokeResponse.Result.IsInvocationSuccess(context.ResultSource, capabilityEnabled))
965965
{
966-
if (FeatureFlags.IsEnabled(ScriptConstants.FeatureFlagEnableHttpProxying) && IsHttpProxyingWorker)
966+
if (IsHttpProxyingWorker)
967967
{
968968
if (context.Properties.TryGetValue(ScriptConstants.HttpProxyTask, out Task<ForwarderError> httpProxyTask))
969969
{

src/WebJobs.Script/ScriptConstants.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,6 @@ public static class ScriptConstants
126126
public const string FeatureFlagEnableWorkerIndexing = "EnableWorkerIndexing";
127127
public const string FeatureFlagEnableDebugTracing = "EnableDebugTracing";
128128
public const string FeatureFlagEnableProxies = "EnableProxies";
129-
public const string FeatureFlagEnableHttpProxying = "EnableHttpProxying";
130129
public const string HostingConfigDisableLinuxAppServiceDetailedExecutionEvents = "DisableLinuxExecutionDetails";
131130
public const string HostingConfigDisableLinuxAppServiceExecutionEventLogBackoff = "DisableLinuxLogBackoff";
132131
public const string FeatureFlagEnableLegacyDurableVersionCheck = "EnableLegacyDurableVersionCheck";

0 commit comments

Comments
 (0)