search

quarkusio / quarkus

Quarkus: Supersonic Subatomic Java.
https://quarkus.io
Apache License 2.0
13.85k stars 2.7k forks source link

Management interface - TLS detection apparently doesn't check TLS registry config #44643

Closed gsmet closed 21 hours ago

gsmet commented 4 days ago

I don't think NonApplicationRootPathBuildItem#isTLsConfigured() has been updated since the TLS registry was introduced.

This code probably needs some updating.

/cc @cescoffier

quarkus-bot[bot] commented 4 days ago

/cc @radcortez (config)

cescoffier commented 4 days ago

Yes, I forgot this direct usage of the Configuration API.

gsmet commented 4 days ago

FWIW, I have a small patch for it in my branch, together with some doc improvements, given I was working in this area.