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

exposed TLS SNI extension #40

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

exposed TLS SNI extension #40

wants to merge 13 commits into from

Conversation

jb0n
Copy link

@jb0n jb0n commented Oct 13, 2015

This change allows a server using spacemonkeygo/openssl to get the TLS extension server name in the standard openssl way (a hostname specified by clients).

@jtolio
Copy link
Member

jtolio commented Nov 29, 2015

Would it be okay to squash all of these together and rebase?

I think there's a chance some of this is already implemented now too

@jb0n
Copy link
Author

jb0n commented Dec 1, 2015

Sure. I'll send another pr squashed. I think I implemented SSL_peek as well
since then because I needed it. It's pretty straight forward and based of
SSL_read.
I didn't see any of the SNI stuff, but I guess I may not have since I
forked. I'll go check it out and see what the current state of things are.

-j

On Sun, Nov 29, 2015 at 10:54 AM, JT Olds [email protected] wrote:

Would it be okay to squash all of these together and rebase?

I think there's a chance some of this is already implemented now too


Reply to this email directly or view it on GitHub
#40 (comment).

jb0n and others added 4 commits September 2, 2016 18:08
…me weird stuff going on working in different dirs (go get vs an actual checkout). This takes care of the go1.6+ pointer ownership/passing rules we were in violation of
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