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

Set certificate expiration to August 2, 2018 #1712

Merged
merged 1 commit into from Aug 3, 2015

Conversation

kevinburke
Copy link
Contributor

Updates the gen-* scripts to take a -days argument, and set that days
argument to 3 years from now. I believe without this argument the certificates
default to a 30 day expiry. Code lifted from
https://github.com/shazow/urllib3/blob/master/dummyserver/certs/README.rst.

Generates new SSL certs with a long expiry.

Updates the gen-* scripts to take a `-days` argument, and set that `days`
argument to 3 years from now. I believe without this argument the certificates
default to a 30 day expiry. Code lifted from
https://github.com/shazow/urllib3/blob/master/dummyserver/certs/README.rst.

Generates new SSL certs with a long expiry.
@kevinburke
Copy link
Contributor Author

whoops... need to check if this passes tests. 1 minute.

@kevinburke kevinburke closed this Aug 3, 2015
@simov
Copy link
Member

simov commented Aug 3, 2015

Something isn't right with our tests lately, it's on my TODO list to investigate why they timeout so badly on TravisCI.

@kevinburke kevinburke reopened this Aug 3, 2015
@kevinburke
Copy link
Contributor Author

When I opened the PR the branches hung on npm install (not sure why). Triggered a rebuild and it looks like the tests are all passing.

simov added a commit that referenced this pull request Aug 3, 2015
Set certificate expiration to August 2, 2018
@simov simov merged commit cf757e1 into request:master Aug 3, 2015
@simov
Copy link
Member

simov commented Aug 3, 2015

Thanks, now you can pull master 👍

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.

None yet

2 participants