We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Hi,
In OwinRequestMapper (line 55) you are setting RequestMessage.DateTime to DateTime.UtcNow:
return new RequestMessage(urldetails, method, clientIP, body, headers, cookies) { DateTime = DateTime.UtcNow };
Then in WireMockMiddleWare (line 175) when you are checking for expiration of that time, you are using DateTime.Now instead of DateTime.UtcNow:
var checkTime = DateTime.Now.AddHours(-_options.RequestLogExpirationDuration.Value);
That's causing you invalid behavior. Please check.
Thanks,
Greg
The text was updated successfully, but these errors were encountered:
#313
117ebc5
Good catch, I've fixed this with PR #314
But you are also allowed to create a PR for other issues you find. ;-)
Sorry, something went wrong.
Thanks @StefH! Perhaps check if there are any other DateTime.Now <> DateTime.UtcNow shenanigans :)
I could not find any other inconsistencies.
#313 (#314)
100291e
StefH
No branches or pull requests
Hi,
In OwinRequestMapper (line 55) you are setting RequestMessage.DateTime to DateTime.UtcNow:
return new RequestMessage(urldetails, method, clientIP, body, headers, cookies) { DateTime = DateTime.UtcNow };
Then in WireMockMiddleWare (line 175) when you are checking for expiration of that time, you are using DateTime.Now instead of DateTime.UtcNow:
var checkTime = DateTime.Now.AddHours(-_options.RequestLogExpirationDuration.Value);
That's causing you invalid behavior.
Please check.
Thanks,
Greg
The text was updated successfully, but these errors were encountered: