eventmesh icon indicating copy to clipboard operation
eventmesh copied to clipboard

[ISSUE #4847] HTTPS/(m)TLSv1.3 support for HTTP Admin Server

Open Pil0tXia opened this issue 1 year ago • 2 comments

Fixes #4847

Motivation

Admin endpoints may need secure transmission.

Modifications

  • Refactor AdminServer to have its own independent configuration class.
  • Start AdminServer even if all three protocols are not enabled.
  • Fix NPE caused by AbstractRemotingServer shutdown() method calling the wrong method.
  • Add TLS support for AdminServer (Document: https://github.com/apache/eventmesh-site/pull/214/files)

Documentation

  • Does this pull request introduce a new feature? (yes)
  • If yes, how is the feature documented? (docs)

Pil0tXia avatar Apr 17 '24 14:04 Pil0tXia

Please fix the pr code conflict problem, and by the way, why do you need to mention pr that supports tls v1.3? jdk8 should not support it, and it needs jdk11 to support it, right? @Pil0tXia

xwm1992 avatar Apr 22 '24 02:04 xwm1992

@xwm1992

I've added a "JDK/JRE 8 Compatibility" chapter in https://github.com/apache/eventmesh-site/pull/214/commits/5ccbf9be142fd0c22f5068b466f4e165140a7a33.

Pil0tXia avatar Apr 22 '24 07:04 Pil0tXia

@pchengma Great

Pil0tXia avatar May 23 '24 19:05 Pil0tXia

@xwm1992 PTAL~

Pil0tXia avatar Jun 01 '24 03:06 Pil0tXia