update undertow and misc reviews

Signed-off-by: neo <1100909+neowu@users.noreply.github.com>

Author: neowu

build.gradle.kts

@@ -39,7 +39,7 @@ project("core-ng") {
         implementation("com.fasterxml.jackson.module:jackson-module-afterburner:${jacksonVersion}")
         implementation("com.fasterxml.jackson.datatype:jackson-datatype-jsr310:${jacksonVersion}")
         implementation("com.squareup.okhttp3:okhttp:5.1.0")
-        implementation("io.undertow:undertow-core:2.3.18.Final")
+        implementation("io.undertow:undertow-core:2.3.19.Final")
         implementation("org.apache.kafka:kafka-clients:4.1.0") {
             exclude("org.xerial.snappy")
             exclude("org.lz4")

core-ng/src/main/java/core/framework/internal/log/LogManager.java

@@ -55,6 +55,8 @@ public static ActionLog currentActionLog() {
     private final ActionLogMessageFactory actionLogMessageFactory = new ActionLogMessageFactory();
     public LogAppender appender;
 
+    // this is internal api, to simplify the design, task must not throw exceptions, all callers catch throwable and log error
+    // if task throws exception, current action log may not be marked as error
     public <T> T run(String action, String id, ActionLogCallable<T> task) {
         var log = new ActionLog("=== " + action + " begin ===", id);
         return where(CURRENT_ACTION_LOG, log).call(() -> {

core-ng/src/test/java/core/framework/internal/web/sse/TestChannelListener.java

@@ -3,10 +3,11 @@
 import core.framework.web.Request;
 import core.framework.web.sse.Channel;
 import core.framework.web.sse.ChannelListener;
+import org.jspecify.annotations.Nullable;
 
 public class TestChannelListener implements ChannelListener<TestEvent> {
     @Override
-    public void onConnect(Request request, Channel<TestEvent> channel, String lastEventId) {
+    public void onConnect(Request request, Channel<TestEvent> channel, @Nullable String lastEventId) {
 
     }
 }

core-ng/src/test/java/core/framework/internal/web/sse/package-info.java

@@ -0,0 +1,4 @@
+@NullMarked
+package core.framework.internal.web.sse;
+
+import org.jspecify.annotations.NullMarked;