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

Fix nil timeout and retries #1256

Merged
merged 1 commit into from
Oct 27, 2016
Merged

Fix nil timeout and retries #1256

merged 1 commit into from
Oct 27, 2016

Conversation

alexpop
Copy link
Contributor

@alexpop alexpop commented Oct 26, 2016

@timeout and @retries were always nil when calling SSLShake.hello

Bumped into this while troubleshooting: #1091

@alexpop alexpop added the Type: Bug Feature not working as expected label Oct 26, 2016
Signed-off-by: Alex Pop <apop@chef.io>
@chris-rock
Copy link
Contributor

Thanks @alexpop for this fix!

@chris-rock chris-rock merged commit efb90c1 into master Oct 27, 2016
@chris-rock chris-rock deleted the ap/ssl-hang branch October 27, 2016 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Feature not working as expected
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants