Skip to content

Commit 258a4db

Browse files
Merge pull request #21427 from abpframework/CorrelationIdProvider
Skip to set `CorrelationId ` if current value is `null`.
2 parents cf41368 + d49665c commit 258a4db

File tree

3 files changed

+17
-3
lines changed

3 files changed

+17
-3
lines changed

framework/src/Volo.Abp.EventBus/Volo/Abp/EventBus/Distributed/DistributedEventBusBase.cs

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,13 @@ protected virtual async Task<bool> AddToOutboxAsync(Type eventType, object event
132132
Serialize(eventData),
133133
Clock.Now
134134
);
135-
outgoingEventInfo.SetCorrelationId(CorrelationIdProvider.Get()!);
135+
136+
var correlationId = CorrelationIdProvider.Get();
137+
if (correlationId != null)
138+
{
139+
outgoingEventInfo.SetCorrelationId(correlationId);
140+
}
141+
136142
await eventOutbox.EnqueueAsync(outgoingEventInfo);
137143
return true;
138144
}

framework/src/Volo.Abp.Http.Client/Volo/Abp/Http/Client/ClientProxying/ClientProxyBase.cs

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -309,7 +309,11 @@ protected virtual void AddHeaders(
309309
}
310310

311311
//CorrelationId
312-
requestMessage.Headers.Add(AbpCorrelationIdOptions.Value.HttpHeaderName, CorrelationIdProvider.Get());
312+
var correlationId = CorrelationIdProvider.Get();
313+
if (correlationId != null)
314+
{
315+
requestMessage.Headers.Add(AbpCorrelationIdOptions.Value.HttpHeaderName, correlationId);
316+
}
313317

314318
//TenantId
315319
if (CurrentTenant.Id.HasValue)

framework/src/Volo.Abp.Http.Client/Volo/Abp/Http/Client/DynamicProxying/ApiDescriptionFinder.cs

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,11 @@ protected virtual async Task<ApplicationApiDescriptionModel> GetApiDescriptionFr
126126
protected virtual void AddHeaders(HttpRequestMessage requestMessage)
127127
{
128128
//CorrelationId
129-
requestMessage.Headers.Add(AbpCorrelationIdOptions.HttpHeaderName, CorrelationIdProvider.Get());
129+
var correlationId = CorrelationIdProvider.Get();
130+
if (correlationId != null)
131+
{
132+
requestMessage.Headers.Add(AbpCorrelationIdOptions.HttpHeaderName, correlationId);
133+
}
130134

131135
//TenantId
132136
if (CurrentTenant.Id.HasValue)

0 commit comments

Comments
 (0)