discourse / mini_mime

minimal mime type library
MIT License
95 stars 34 forks source link

Markdown, MD return wrong content type. #12

Closed envygeeks closed 5 years ago

envygeeks commented 6 years ago

Even though this has been filed upstream, they've been slow to progress on the issue, even though an RFC exists for text/markdown being md, and markdown, and a pull-request exists upstream (see: https://github.com/mime-types/mime-types-data/issues/15, https://github.com/mime-types/mime-types-data/pull/13) this problem has yet to be fixed, so I'm hoping we can get a quick resolution here?

SamSaffron commented 6 years ago

@jeremy what are your thoughts here, on one hand it is an easy one line change, but on the other hand we all agree it belongs upstream.

jeremy commented 6 years ago

Agree we should take care to track upstream, one-liner temptations aside 😄

Looks like the PR is essentially a bump from IANA upstream, suggesting that mime-types-data could use a more automated release process to keep up to date. Good way to contribute in permanently effective way?

/cc @halostatue

halostatue commented 6 years ago

Yes, I would happily consider PRs that improve the automation of mime-types-data, or even possibly finding a maintainer who has more time (most of my development time these days is not currently in Ruby).

SamSaffron commented 5 years ago

Yay, I just pulled it in from upstream going to do a new 1.0.1 version right now

halostatue commented 5 years ago

Cool. One thing to point out is that extensions are not currently available from the raw IANA data. If they were, I would happily extend the parser.

envygeeks commented 5 years ago

I noticed that a couple of weeks ago when I was trying to update some sys mimetypes on Linux in an automated way and I had to do it all manually because they mix documentation and extensions >.> it was easier to just parse the HTML from Wikipedia and extract it from there.