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.

Sign up for GitHub

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

Jump to bottom

refresh SSL certs early #222

Merged
merged 1 commit into from
Oct 31, 2018
Merged

refresh SSL certs early #222

merged 1 commit into from
Oct 31, 2018

Conversation

lychung83
Copy link
Contributor

@lychung83 lychung83 commented Oct 29, 2018 •
edited

fixes #94. Review done internally.

@kurtisvg kurtisvg added priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. Status: Review Needed type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. labels Oct 29, 2018
@lychung83
Copy link
Contributor Author

Did manual testing for making iterative connections (and closing after short period of time) for 205 hours, but no high latency except for the first connection and there's also no connection failure.

@lychung83 lychung83 merged commit 1214300 into GoogleCloudPlatform:master Oct 31, 2018
@lychung83 lychung83 mentioned this pull request Oct 31, 2018
Closed
yosatak pushed a commit to yosatak/cloud-sql-proxy that referenced this pull request Feb 26, 2023
@dazuma
feat: Update Ruby versions (GoogleCloudPlatform#222)
39c67c7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@easwars easwars easwars approved these changes

Assignees

@hfwang hfwang

Labels
priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Feature: Refresh SSL cert before it expires in the background
4 participants
@lychung83 @easwars @hfwang @kurtisvg