Skip to content
New issue

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

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

Already on GitHub? # to your account

Upgrade json dependency in view of CVE-2020-10663 #2

Closed
wants to merge 1 commit into from

Conversation

francois
Copy link

@francois francois commented Jun 1, 2020

@killthekitten
Copy link
Owner

Hey @francois, thanks for the patch! I believe this is only a mirror that hasn't been updated in a while. Could you send the PR to the original repo? https://bitbucket.org/mailchimp/mandrill-api-ruby/src/master/

@francois francois closed this Jun 19, 2020
@makstaks
Copy link

makstaks commented Aug 10, 2020

Unfortunately, the gem is no longer supported see: https://bitbucket.org/mailchimp/mandrill-api-ruby/pull-requests/8/fix-json-version/diff

Thank you for writing in, I apologize for the delay in getting back to you. Currently, we do not have any official support for those public libraries any longer. We are more than happy to pass along the feedback to our developers, but I can not make promises that there will be any updates. However, as the code is public, you are entirely welcome to manipulate and edit it as needed to work with newer version of Python. I understand that this is not the ideal situation, but I am not privy to any information of upcoming updates to our Ruby wrappers.

@killthekitten
Copy link
Owner

@makstaks that's a pitty. I would recommend to move away from using mailchimp if that's possible, or use @francois patch as a github dependency in your Gemfile.

@1wilber
Copy link

1wilber commented Jun 29, 2023

@killthekitten the repository was deleted, what version of json are compatible with this gem? i want fork this repo and upgrade de json version

@killthekitten
Copy link
Owner

@NotB0T not sure exactly, but judging from the gemspec anything fresh should work (also without requiring an upgrade):

s.add_dependency 'json', '>= 1.7.7'

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants