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

Try and make TLS federation client code faster #4674

Merged
merged 3 commits into from Feb 19, 2019

Conversation

Projects
None yet
3 participants
@hawkowl
Copy link
Contributor

hawkowl commented Feb 19, 2019

Fixes #4673

@hawkowl hawkowl requested a review from matrix-org/synapse-core Feb 19, 2019

@codecov-io

This comment has been minimized.

Copy link

codecov-io commented Feb 19, 2019

Codecov Report

Merging #4674 into develop will increase coverage by 0.03%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #4674      +/-   ##
===========================================
+ Coverage    75.17%   75.21%   +0.03%     
===========================================
  Files          340      340              
  Lines        34712    34712              
  Branches      5677     5677              
===========================================
+ Hits         26096    26107      +11     
+ Misses        7013     7004       -9     
+ Partials      1603     1601       -2
@@ -0,0 +1 @@
Reduce the overhead of creating outbound federation connections over TLS by caching the TLS client options.

This comment has been minimized.

@richvdh

richvdh Feb 19, 2019

Member

.feature, maybe? I think people should be excited about it

@richvdh
Copy link
Member

richvdh left a comment

lgtm

@richvdh richvdh merged commit 7c70b8f into develop Feb 19, 2019

0 of 5 checks passed

ci/circleci: sytestpy2merged Your tests are queued behind your running builds
Details
ci/circleci: sytestpy2postgresmerged Your tests are queued behind your running builds
Details
ci/circleci: sytestpy3merged Your tests are queued behind your running builds
Details
ci/circleci: sytestpy3postgresmerged Your tests are queued behind your running builds
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.