Skip to content

[ca/manager] Remove root CA key encryption support entirely#2573

Merged
dperny merged 1 commit into
moby:masterfrom
cyli:blow-up-root-ca-encryption
Mar 27, 2018
Merged

[ca/manager] Remove root CA key encryption support entirely#2573
dperny merged 1 commit into
moby:masterfrom
cyli:blow-up-root-ca-encryption

Conversation

@cyli

@cyli cyli commented Mar 27, 2018

Copy link
Copy Markdown
Contributor

Since that feature, neither documented nor advertised, was deprecated almost a year ago. Rely on MTLS for encryption in transit and raft log encryption for encryption at rest.

@cyli cyli force-pushed the blow-up-root-ca-encryption branch from 3b0d1e5 to ea2940e Compare March 27, 2018 00:41
that feature was deprecated almost a year ago.  Rely on MTLS for
encryption in transit and raft log encryption for encryption at rest.

Signed-off-by: Ying Li <ying.li@docker.com>
@codecov

codecov Bot commented Mar 27, 2018

Copy link
Copy Markdown

Codecov Report

Merging #2573 into master will decrease coverage by 9.32%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2573      +/-   ##
==========================================
- Coverage    61.4%   52.07%   -9.33%     
==========================================
  Files          49       25      -24     
  Lines        6332     2669    -3663     
==========================================
- Hits         3888     1390    -2498     
+ Misses       2067     1136     -931     
+ Partials      377      143     -234
@dperny

dperny commented Mar 27, 2018

Copy link
Copy Markdown
Collaborator

LGTM

@dperny dperny merged commit 876aec9 into moby:master Mar 27, 2018
@cyli cyli deleted the blow-up-root-ca-encryption branch March 27, 2018 21:50
@cyli cyli mentioned this pull request May 7, 2018
@cyli cyli mentioned this pull request Jun 7, 2018
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants