Deprecate PKCS#1 (#1368) #1590
Merged
Deprecate PKCS#1 (#1368) #1590
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
adleong
reviewed
Aug 17, 2017
| // We should remove this and replace it with Netty4ServerEngineFactory once we no longer allow | ||
| // PKCS#1 keys. | ||
| @JsonIgnore | ||
| @silent |
| // We should remove this and replace it with Netty4ServerEngineFactory once we no longer allow | ||
| // PKCS#1 keys. | ||
| @JsonIgnore | ||
| @silent |
PKCS#1 requires LegacyKeyServerEngineFactory, which shells out top openssl, which in turn requires library/java:openjdk-8-jre as a base Docker image. We'd like to move to library/openjdk:8-jre-alpine. Remove support for PKCS#1, replace LegacyKeyServerEngineFactory with Netty4ServerEngineFactory. Validated with unit tests, example configs, and integration test suite. fixes #1368
f143fb3 to
59ed0a3
Compare
hawkw
added a commit
that referenced
this pull request
Sep 7, 2017
## 1.2.0 2017-09-07 * **Breaking Change**: `io.l5d.mesh`, `io.l5d.thriftNameInterpreter`, linkerd admin, and namerd admin now serve on 127.0.0.1 by default (instead of 0.0.0.0). * **Breaking Change**: Removed support for PKCS#1-formatted keys. PKCS#1 formatted keys must be converted to PKCS#8 format. * Added experimental `io.l5d.dnssrv` namer for DNS SRV records (#1611) * Kubernetes * Added an experimental `io.l5d.k8s.configMap` interpreter for reading dtabs from a Kubernetes ConfigMap (#1603). This interpreter will respond to changes in the ConfigMap, allowing for dynamic dtab updates without the need to run Namerd. * Made ingress controller's ingress class annotation configurable (#1584). * Fixed an issue where Linkerd would continue routing traffic to endpoints of a service after that service was removed (#1622). * Major refactoring and performance improvements to `io.l5d.k8s` and `io.l5d.k8s.ns` namers (#1603). * Ingress controller now checks all available ingress resources before using a default backend (#1607). * Ingress controller now correctly routes requests with host headers that contain ports (#1607). * HTTP/2 * Fixed an issue where long-running H2 streams would eventually hang (#1598). * Fixed a memory leak on long-running H2 streams (#1598) * Added a user-friendly error message when a HTTP/2 router receives a HTTP/1 request (#1618) * HTTP/1 * Removed spurious `ReaderDiscarded` exception logged on HTTP/1 retries (#1609) * Consul * Added support for querying Consul by specific service health states (#1601) * Consul namers and Dtab store now fall back to a last known good state on Consul observation errors (#1597) * Improved log messages for Consul observation errors (#1597) * TLS * Removed support for PKCS#1 keys (#1590) * Added validation to prevent incompatible `disableValidation: true` and `clientAuth` settings in TLS client configurations (#1621) * Changed `io.l5d.mesh`, `io.l5d.thriftNameInterpreter`, linkerd admin, and namerd admin to serve on 127.0.0.1 by default (instead of 0.0.0.0) (#1366) * Deprecated `io.l5d.statsd` telemeter.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PKCS#1 requires LegacyKeyServerEngineFactory, which shells out top
openssl, which in turn requires library/java:openjdk-8-jre as a base
Docker image. We'd like to move to library/openjdk:8-jre-alpine.
Remove support for PKCS#1, replace LegacyKeyServerEngineFactory with
Netty4ServerEngineFactory.
Validated with unit tests, example configs, and integration test suite.
fixes #1368