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

Add TLS cluster support (using my unfinished contribution) #99

Open
aasaru opened this issue Jul 2, 2020 · 2 comments
Open

Add TLS cluster support (using my unfinished contribution) #99

aasaru opened this issue Jul 2, 2020 · 2 comments

Comments

@aasaru
Copy link

aasaru commented Jul 2, 2020

Hi!

Great library, thanks for that. I wanted to make a pull request that adds TLS support.
I did all the changes to code and documentation but unfortunately I have some little issue with OpenSSL that results with error "Error accepting a client connection: error:1408F10B:SSL routines:ssl3_get_record:wrong version number"

I wonder if anyone would be willing to look at this and maybe get it finished. If not then feel free to close this issue.

My work is here:
master...aasaru:tls
Feel free to fork it.

@Grokzen
Copy link
Owner

Grokzen commented Jul 2, 2020

@aasaru If you think the work is good enough, make a PR and i can review and comment on it, and if it looks good then i can merge it in and make new builds available with your contributions in it.

@aasaru
Copy link
Author

aasaru commented Jul 6, 2020

Thanks! The problem is that I think i did all the required changes but for some reason it didn't work and I gave up. Instead of throwing all my work away I thought I let you know. Maybe there will be someone who will eventually pick this up what I started and finish it.

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

No branches or pull requests

2 participants