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

Address comments by Christer Holmberg. #50

merged 4 commits into from Nov 4, 2019


Copy link

grittygrease commented Sep 18, 2019

No description provided.

grittygrease and others added 2 commits Sep 18, 2019
Nick Sullivan Nick Sullivan
@grittygrease grittygrease requested review from chris-wood and ekr Oct 10, 2019
Copy link

martinthomson left a comment

I think that this is sneaking in a big change with the DTLS thing. I'll try to find the email... Outdated Show resolved Hide resolved Outdated Show resolved Hide resolved
grittygrease and others added 2 commits Oct 15, 2019
Co-Authored-By: Martin Thomson <>

This comment has been minimized.

Copy link
Collaborator Author

grittygrease commented Oct 15, 2019

@martinthomson are you happy with the adjusted text? I don't consider DTLS support to be a big change since the only thing we are relying on from TLS is the exporter interface.

@grittygrease grittygrease merged commit ad8ea71 into master Nov 4, 2019
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
continuous-integration/travis-ci/push The Travis CI build passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
3 participants
You can’t perform that action at this time.