AzureAD/microsoft-identity-abstractions-for-dotnet

[Feature Request] Does AcquireTokenOptions.CorrelationId need to be a Guid?

keegan-caruso opened this issue · 2 comments

keegan-caruso commented

Does the AcquireTokenOptions.CorrelationId need to be a Guid or would a string be sufficient?

If not set, then the default value for this will be the default value of a Guid, all zeros.

Is it strictly required that the correlation ID being a GUID is another question

jmprieur commented

@keegan-caruso : le'ts have a string. It allows for hierarchical correlation ids (that ASP.NET Core uses)
@jennyf19 let's replace the GUID by a string in our next release:

microsoft-identity-abstractions-for-dotnet/src/Microsoft.Identity.Abstractions/TokenAcquisition/AcquireTokenOptions.cs

Line 66 in f77360e

public Guid CorrelationId { get; set; }

jennyf19 commented

released in 1.0.0-preview.