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 for NPE when the OpenSSLSocketImpl returns null from getNpnSelectedProtocol #513

Merged
merged 1 commit into from Feb 6, 2014

Conversation

3 participants
@adamsp
Contributor

adamsp commented Feb 5, 2014

I detailed this issue here when trying to track down a different issue (which I'm working through with Jesse on StackOverflow).

@adriancole

This comment has been minimized.

Show comment
Hide comment
@adriancole

adriancole Feb 5, 2014

Collaborator

Great, @adamsp. Do you have a contributor agreement on file?

Collaborator

adriancole commented Feb 5, 2014

Great, @adamsp. Do you have a contributor agreement on file?

@JakeWharton

This comment has been minimized.

Show comment
Hide comment
@JakeWharton

JakeWharton Feb 5, 2014

Collaborator

He does.

Collaborator

JakeWharton commented Feb 5, 2014

He does.

@adamsp

This comment has been minimized.

Show comment
Hide comment
@adamsp

adamsp Feb 5, 2014

Contributor

@adriancole signed it just after I sent this PR :)

Contributor

adamsp commented Feb 5, 2014

@adriancole signed it just after I sent this PR :)

adriancole added a commit that referenced this pull request Feb 6, 2014

Merge pull request #513 from adamsp/master
Fix for NPE when the OpenSSLSocketImpl returns null from getNpnSelectedProtocol

@adriancole adriancole merged commit 08b68ac into square:master Feb 6, 2014

1 check passed

default The Travis CI build passed
Details
@adriancole

This comment has been minimized.

Show comment
Hide comment
@adriancole

adriancole Feb 6, 2014

Collaborator

Thx!

Collaborator

adriancole commented Feb 6, 2014

Thx!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment