Skip to content
This repository was archived by the owner on Nov 20, 2018. It is now read-only.

Fix for https://github.com/aspnet/Mvc/issues/4883 #658

Closed
wants to merge 2 commits into from
Closed

Fix for https://github.com/aspnet/Mvc/issues/4883 #658

wants to merge 2 commits into from

Conversation

Haplois
Copy link

@Haplois Haplois commented Jun 19, 2016

Summary of the changes

  • Fixed parsing of header values containing consecutive commas.
  • Created a test for fix.

Addresses aspnet/Mvc#4883

@dnfclas
Copy link

dnfclas commented Jun 19, 2016

Hi @Haplois, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.

TTYL, DNFBOT;

@@ -26,5 +26,33 @@ public void PropertiesAreAccessible()
Assert.Equal("Value1", headers["header1"]);
Assert.Equal(new[] { "Value1" }, headers["header1"].ToArray());
}

[Fact]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can this test be written as a Theory?

@Haplois
Copy link
Author

Haplois commented Jun 19, 2016

@Tratcher I just refactored it to a Theory.

@Tratcher Tratcher self-assigned this Jun 21, 2016
@Tratcher Tratcher modified the milestones: 1.0.1, 1.1.0 Jun 23, 2016
@Tratcher
Copy link
Member

Merged, thanks.

@Tratcher Tratcher closed this Jun 23, 2016
@muratg muratg modified the milestones: 1.1.0, 1.0.1 Jun 29, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants