SCIMReferenceCode icon indicating copy to clipboard operation
SCIMReferenceCode copied to clipboard

RequestExtensions.TryGetRequestIdentifier and overall usage of "correlationIdentifier"

Open norfas opened this issue 2 years ago • 0 comments

The code from RequestExtensions class quoted below does not make sense, since request?.Headers.TryGetValues(...) call is a "dead code" that has no effect. And judging by this code I doubt that "correlationIdentifier" is being used to any meaningful purpose in this project.

public static bool TryGetRequestIdentifier(this HttpRequestMessage request, out string requestIdentifier) { request?.Headers.TryGetValues("client-id", out IEnumerable<string> _); requestIdentifier = Guid.NewGuid().ToString(); return true; }

Being used like this: if (!request.TryGetRequestIdentifier(out correlationIdentifier)) { throw new HttpResponseException(HttpStatusCode.InternalServerError); }

norfas avatar Feb 01 '23 14:02 norfas