Skip to content

Added ClientSessionCache and SessionTicketsDisabled TLS client options#139

Merged
fredbi merged 1 commit intogo-openapi:masterfrom
fredbi:feat-add-client-tls-cache
Jun 20, 2019
Merged

Added ClientSessionCache and SessionTicketsDisabled TLS client options#139
fredbi merged 1 commit intogo-openapi:masterfrom
fredbi:feat-add-client-tls-cache

Conversation

@fredbi
Copy link
Copy Markdown
Member

@fredbi fredbi commented Apr 29, 2019

I believe that today's slack discussion on TLS client config may be resolved by something as simple as this. But maybe that is too simple?

Signed-off-by: Frederic BIDON fredbi@yahoo.com

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 29, 2019

Codecov Report

Merging #139 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #139      +/-   ##
==========================================
+ Coverage   80.86%   80.87%   +0.01%     
==========================================
  Files          40       40              
  Lines        2294     2296       +2     
==========================================
+ Hits         1855     1857       +2     
  Misses        337      337              
  Partials      102      102
Impacted Files Coverage Δ
client/runtime.go 64.21% <100%> (+0.35%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f4da4bf...bb3e3d6. Read the comment docs.

@casualjim
Copy link
Copy Markdown
Member

Can you update?

Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
@fredbi fredbi force-pushed the feat-add-client-tls-cache branch from da39b9c to bb3e3d6 Compare June 20, 2019 22:12
@fredbi fredbi merged commit 4c4e27b into go-openapi:master Jun 20, 2019
@fredbi fredbi deleted the feat-add-client-tls-cache branch July 4, 2019 22:27
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