Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add dependency for mime-types v <3 #32

Merged
merged 1 commit into from
Sep 30, 2016

Conversation

rogiervandenberg
Copy link
Contributor

After installing the mollie-api-ruby gem I was not able to use the API, due to an "undefined method `[]' for #<Set: {#<MIME::Type: application/json>}>" error. I found out it was because of my version of mime-types (3.1).

This fix forces the use of mime-types < v3

See also: https://twitter.com/RogiervdBerg/status/768184731284414466

Version 2.99 of mime-types is still supported as 2.99.x, receiving quarterly updates of the IANA registry and security updates for two years. It will reach full end of life on 2017-11-21.

@ndijkstra
Copy link

Hi Rogier! Thanks for your Pull Request.

@Daanvm
Copy link
Contributor

Daanvm commented Aug 26, 2016

Hi Rogier,
Thanks for the PR!

I'm not really a ruby developer myself, so correct me if I'm wrong, but wouldn't it be better to fix this issue by adding support for the newer mime-types gem?

@rogiervandenberg
Copy link
Contributor Author

Hi Daan, yes perhaps, but I think that should be solved within the REST Client gem. So to make sure your gem works, independently of REST Client, this might help. I am not sure whether this is a good solution though, it is more a 'quick fix'.

Copy link
Contributor

@Daanvm Daanvm left a comment

Choose a reason for hiding this comment

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

Thanks again for the PR. I think this 'quick fix' is the right solution for our current Ruby client.

We're working on a new 2.0.0 release, which should support the latest version.

@Daanvm Daanvm merged commit a1fea67 into mollie:master Sep 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants