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
Add support for a Spring Boot custom key alias #1865
Conversation
40c98fd
to
6ecc7fd
Compare
0e6527a
to
951371e
Compare
Will add a test case early next week. |
Codecov Report
@@ Coverage Diff @@
## master #1865 +/- ##
============================================
+ Coverage 73.02% 73.25% +0.22%
- Complexity 8823 8868 +45
============================================
Files 784 786 +2
Lines 34670 34731 +61
Branches 4233 4242 +9
============================================
+ Hits 25319 25441 +122
+ Misses 7199 7128 -71
- Partials 2152 2162 +10
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work! @trustin
...toconfigure/src/main/java/com/linecorp/armeria/internal/spring/ArmeriaConfigurationUtil.java
Show resolved
Hide resolved
spring/boot-autoconfigure/src/main/java/com/linecorp/armeria/spring/ArmeriaSettings.java
Show resolved
Hide resolved
spring/boot-autoconfigure/src/main/java/com/linecorp/armeria/spring/Ssl.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work!
Motivation: The `ssl.keyAlias` configuration in Spring Boot is currently unsupported. Modifications: - Add `CustomAliasKeyManagerFactory` and `CustomAliasX509ExtendedKeyManager` - Wrap the `KeyManagerFactory` with `CustomAliasKeyManagerFactory` to support custom key alias - Miscellaneous: - Add `@Nullable` annotations where necessary Result: - Fixes line#1843
951371e
to
374bc00
Compare
@anuraaga You might want to check this out as well. |
...va/com/linecorp/armeria/spring/web/reactive/AbstractReactiveWebServerCustomKeyAliasTest.java
Outdated
Show resolved
Hide resolved
Motivation: The `ssl.keyAlias` configuration in Spring Boot is currently unsupported. Modifications: - Add `CustomAliasKeyManagerFactory` and `CustomAliasX509ExtendedKeyManager` - Wrap the `KeyManagerFactory` with `CustomAliasKeyManagerFactory` to support custom key alias - Miscellaneous: - Add `@Nullable` annotations where necessary Result: - Fixes line#1843
Motivation:
The
ssl.keyAlias
configuration in Spring Boot is currentlyunsupported.
Modifications:
CustomAliasKeyManagerFactory
andCustomAliasX509ExtendedKeyManager
KeyManagerFactory
withCustomAliasKeyManagerFactory
tosupport custom key alias
@Nullable
annotations where necessaryResult: