Skip to content
This repository has been archived by the owner. It is now read-only.

Adding support for media range parameters #1259

Merged
merged 1 commit into from Oct 10, 2013
Merged

Conversation

@liddellj
Copy link
Contributor

@liddellj liddellj commented Oct 10, 2013

Fix for #1211.

  • Added a new class called MediaParameters which implements IEnumerable<KeyValuePair<string, string>>
  • Added a Parameters property to MediaRange, of the above type.
  • Altered MediaRange.FromString to parse the parameters into the new property.
  • Added MediaRange.MatchesWithParameters to take parameters into account (if parameters exist, they must exist in both, but ordering doesn't matter).
  • Modify RequestHeaders.GetWeightedValues so that media range parameters are not stripped.
grumpydev added a commit that referenced this pull request Oct 10, 2013
Adding support for media range parameters
@grumpydev grumpydev merged commit 9d2b9fb into NancyFx:master Oct 10, 2013
1 check passed
1 check passed
default The Travis CI build passed
Details
@liddellj liddellj deleted the liddellj:mediarange-params branch Oct 10, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.