SyliusResourceBundle icon indicating copy to clipboard operation
SyliusResourceBundle copied to clipboard

Improved API for SSL context management (server and client)

Open DerGuteMoritz opened this issue 1 year ago • 3 comments

This is a cherry-pick of dac2852c73c8ceeecaa15549c9de08dbcd530efd from the 1.0.0 adjusted to latest master and with some additions and corrections to docstrings.

As per https://github.com/clj-commons/aleph/pull/608#issuecomment-1200103587

DerGuteMoritz avatar Jul 30 '22 20:07 DerGuteMoritz

Regarding TLS, an improvement would be to support javax.net.ssl SSLContext out of the box.

@arnaudgeiser Seems like this should be doable via https://netty.io/4.1/api/io/netty/handler/ssl/JdkSslContext.html which one could then pass as :ssl-context. Some builtin support for this in Aleph would be more convenient, of course. Should be easy to do now by adding it to coerce-ssl-context. Ideally with a test case, of course... I suggest we make this a separate issue?

DerGuteMoritz avatar Aug 05 '22 15:08 DerGuteMoritz

I suggest we make this a separate issue?

Yes, obviously. Just surprised it wasn't supported already.

arnaudgeiser avatar Aug 06 '22 12:08 arnaudgeiser

@KingMob FYI: I'll resolve the conflicts (and squash where appropriate) once you've approved my latest changes!

DerGuteMoritz avatar Aug 09 '22 10:08 DerGuteMoritz

I tried to rebase this PR on master but unfortunately I don't have the rights to perform commits on your branch @DerGuteMoritz. Can I ask you to do so then I will merge?

arnaudgeiser avatar Aug 15 '22 11:08 arnaudgeiser

@arnaudgeiser There you go! Only squashed the indentation commit after all.

DerGuteMoritz avatar Aug 17 '22 13:08 DerGuteMoritz

Hold on a moment, I accidentlly clobbered Oleksii's original commit authorship, restoring ...

DerGuteMoritz avatar Aug 17 '22 13:08 DerGuteMoritz

Alright, good to go now from my end!

DerGuteMoritz avatar Aug 17 '22 13:08 DerGuteMoritz