This repository has been archived by the owner on Sep 18, 2021. It is now read-only.

ConfigureHttpLogging needs some cleanup on stream #3779

Open

brockallen opened this issue Jul 13, 2017 · 0 comments

Labels

Member

brockallen commented Jul 13, 2017

aspnet/AspNetKatana#51 (comment)

@brockallen
When I copied ConfigureHttpLogging for debugging I found it was missing a Seek on the response body before CopyToAsync. It could also use a Length check. See above.

IdentityServer3/source/Core/Configuration/AppBuilderExtensions/ConfigureHttpLoggingExtension.cs

Line 48 in 93bc6bc

await ms.CopyToAsync(oldStream);

brockallen added the bug report label

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.