jshttp / mime-types

The ultimate javascript content-type utility.
MIT License
1.32k stars 129 forks source link

Default to the most recently added definition for mime.extension(type) #5

Closed Fishrock123 closed 10 years ago

Fishrock123 commented 10 years ago

I feel as though mime.extension should return the last added definition for the type.

Thoughts?

jonathanong commented 10 years ago

i think the first is better. you'll have to look at the types specifically to see. stuff defined first is the apache stuff, and those are probably the most common ones

Fishrock123 commented 10 years ago

Not necessarily for though. Think application/octet-stream - which in the context of node are probably* buffers.

*(imo quite likely)

jonathanong commented 10 years ago

if someone is doing extensions('application/octet-stream') then they probably have bigger problems