-
Notifications
You must be signed in to change notification settings - Fork 644
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Propagate correlation id from gallery to search service #2892
Conversation
Hi @skofman1, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution! TTYL, DNFBOT; |
@maartenba, @xavierdecoster , is this ok to submit? |
@skofman1, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR. |
@@ -169,8 +174,8 @@ public IHttpActionResult GetPropertyFromPackages(string propertyName, string id, | |||
{ | |||
switch (propertyName.ToLowerInvariant()) | |||
{ | |||
case "id": return Ok(id); | |||
case "version": return Ok(version); | |||
case "id": return Ok(id); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whitespace seems gone?
Minor comment (on adding SetCorrelation() to the constructor). Other than that looks good to me, :) |
Propagate correlation id from gallery to search service
No description provided.