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

Update CA bundle and add script for future updates #421

Merged
merged 2 commits into from
Jan 17, 2018

Conversation

ob-stripe
Copy link
Contributor

r? @brandur-stripe
cc @stripe/api-libraries

Updates the CA bundle to the latest version, and adds a script to make future updates easier.

@brandur-stripe
Copy link
Contributor

Nice!

What do you think about adding a basic section to the README describing how to run the new script? For stripe-ruby, we have this:

Update bundled CA certificates from the [Mozilla cURL release][curl]:

    bundle exec rake update_certs

(Also, a couple of the builds in the matrix failed with what looks like a problem fetching dependencies. I restarted a bunch of them.)

@ob-stripe
Copy link
Contributor Author

ob-stripe commented Jan 17, 2018

What do you think about adding a basic section to the README describing how to run the new script?

👍 Pushed a new commit to update the README.

(Also, a couple of the builds in the matrix failed with what looks like a problem fetching dependencies. I restarted a bunch of them.)

Yeah, packagist.org has been acting up a lot these past few days.

@brandur-stripe brandur-stripe merged commit 58fadb2 into integration-v6 Jan 17, 2018
@brandur-stripe brandur-stripe deleted the ob-update-ca-bundle branch January 17, 2018 20:52
@brandur-stripe
Copy link
Contributor

Released as 5.9.0.

@ob-stripe ob-stripe mentioned this pull request Jan 18, 2018
9 tasks
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.

2 participants