search

jshttp / accepts

Higher-level content negotiation
MIT License
252 stars 42 forks source link

fix unknown extension given in array #7

Closed dead-horse closed 10 years ago

dougwilson commented 10 years ago

Cool. I feel like the changes are more than necessary, but I'll get this in ASAP. What is the current behavior of accept.types('bogus', 'html') without this patch?

dead-horse commented 10 years ago

'bogus' :(

dougwilson commented 10 years ago

'bogus' :(

Weeeird, lol, because that should never come back in the results, haha. Definitely needs to be fixed then.

dougwilson commented 10 years ago

@dead-horse fix should be in now if you want to verify it :)

dead-horse commented 10 years ago

cool

dougwilson commented 10 years ago

Just didn't want to have too many changes so I could release this as a patch version :)

dougwilson commented 10 years ago

Published as accepts@1.0.7