Transaction id should be optional for correlation info (#13)

* Provide unit tests for CorrelationInfo Signed-off-by: Tom Kerkhove <[email protected]> * Fix build issue Signed-off-by: Tom Kerkhove <[email protected]> * Transaction id should not be mandatory
arcus-azure · Feb 5, 2020 · fd67003 · fd67003
1 parent 6915ae0
commit fd67003
Showing 1 changed file with 1 addition and 5 deletions.
diff --git a/src/Arcus.Observability.Correlation/CorrelationInfo.cs b/src/Arcus.Observability.Correlation/CorrelationInfo.cs
@@ -1,5 +1,4 @@
-using System;
-using GuardNet;
+using GuardNet;
 
 namespace Arcus.Observability.Correlation
 {
@@ -15,9 +14,6 @@ public CorrelationInfo(string operationId, string transactionId)
         {
             Guard.NotNullOrEmpty(operationId, nameof(operationId),
                 "Cannot create a correlation instance with a blank operation ID");
-            Guard.For<ArgumentException>(
-                () => transactionId == string.Empty,
-                "Cannot create correlation instance with a blank transaction ID, only 'null' or non-blank ID's are allowed");
 
             OperationId = operationId;
             TransactionId = transactionId;