Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Check that Content-Type is correct before accessing the Form #322

Merged
merged 1 commit into from
Oct 4, 2018
Merged

Check that Content-Type is correct before accessing the Form #322

merged 1 commit into from
Oct 4, 2018

Conversation

mcliment
Copy link
Contributor

@mcliment mcliment commented Oct 4, 2018

This fixes #321

@codecov
Copy link

codecov bot commented Oct 4, 2018 •
edited
Loading

Codecov Report

Merging #322 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #322   +/-   ##
=====================================
  Coverage      62%    62%           
=====================================
  Files          32     32           
  Lines         454    454           
  Branches       97     97           
=====================================
  Hits          282    282           
  Misses        135    135           
  Partials       37     37
Impacted Files Coverage Δ
...LayoutRenderers/AspNetRequestFormLayoutRenderer.cs 84% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7088c9f...38ea8cb. Read the comment docs.

@304NotModified
Copy link
Member

Thanks! Also for the unit test 👍

@304NotModified 304NotModified merged commit 3ebb807 into NLog:master Oct 4, 2018
@mcliment mcliment deleted the fix-content-type-error branch October 5, 2018 07:17
@snakefoot snakefoot mentioned this pull request Oct 28, 2018
Merged
@snakefoot snakefoot added ASP.NET Core ASP.NET Core - all versions and removed ASP.NET Core 1 labels Jan 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ASP.NET Core ASP.NET Core - all versions
Projects
None yet
Milestone
NLog.Web + NLog.Web.AspNetCore 4.7.1
Development

Successfully merging this pull request may close these issues.

${aspnet-request-form} raises exception in ASP.NET Core
3 participants
@mcliment @304NotModified @snakefoot