Skip to content

Conversation

@aegal
Copy link
Contributor

@aegal aegal commented Nov 25, 2020

This change is needed to support recent TLS changes

  1. Read from Encrypted PEM file
  2. Read hostname from CNS commands if provided

@codecov
Copy link

codecov bot commented Nov 25, 2020

Codecov Report

Merging #742 (06dab24) into master (db8724e) will increase coverage by 0.43%.
The diff coverage is 61.11%.

@@            Coverage Diff             @@
##           master     #742      +/-   ##
==========================================
+ Coverage   39.17%   39.61%   +0.43%     
==========================================
  Files          83       82       -1     
  Lines       10834    10800      -34     
==========================================
+ Hits         4244     4278      +34     
+ Misses       6092     6017      -75     
- Partials      498      505       +7     

Copy link
Contributor

@ramiro-gamarra ramiro-gamarra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few comments

Copy link
Contributor

@ramiro-gamarra ramiro-gamarra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@aegal aegal merged commit 169d7d7 into master Nov 30, 2020
@rbtr rbtr deleted the tlsupdate branch November 30, 2021 20:14
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.

3 participants