HTHou opened a new pull request, #18299:
URL: https://github.com/apache/iotdb/pull/18299

   ## Description
   
   This PR adds internal hooks for reporting SSL/TLS channel setup failures 
together with the initiator and target endpoints. It does not add or expose a 
new logging or persistence surface.
   
   ### Failure context
   
   - Add a shared failure callback that propagates the original exception and 
both channel endpoints.
   - Generalize the existing session-specific failure event name for use across 
transport types.
   - Recognize SSL failures through the exception cause chain while preserving 
the original connection failure behavior.
   - Isolate reporting callback failures so they do not replace the transport 
exception.
   
   ### Transport coverage
   
   - Report outbound SSL failures for ConfigNode-to-DataNode, 
DataNode-to-ConfigNode, DataNode-to-DataNode, and DataNode-to-AINode request 
paths.
   - Report inbound TLS handshake failures for ConfigNode RPC, DataNode 
internal RPC, MPP exchange, IoTConsensus V1/V2, and the Session/JDBC-facing RPC 
service.
   - Report Jetty HTTPS handshake failures for the REST service.
   - Keep plaintext paths unchanged and use no-op callbacks when failure 
reporting is not configured.
   
   ### Validation
   
   - `mvn test -pl iotdb-core/node-commons 
-Dtest=AbstractAuditLoggerTest,TrustedChannelAuditServerEventHandlerTest`
   - `mvn test -pl external-service-impl/rest -am 
-Dtest=TrustedChannelAuditHandshakeListenerTest 
-Dsurefire.failIfNoSpecifiedTests=false -DfailIfNoTests=false`
   - `mvn test-compile -DskipTests -pl external-service-impl/rest -am`
   - `mvn test-compile -P with-zh-locale -DskipTests -pl 
external-service-impl/rest -am`
   - `mvn dependency:analyze-only -pl external-service-impl/rest -DskipTests`
   - Spotless, Checkstyle, and `git diff --check`
   
   <hr>
   
   This PR has:
   - [x] been self-reviewed.
   - [x] added Javadocs for the shared callback and server-side handler.
   - [x] added unit tests for the new failure-reporting paths.
   
   <hr>
   
   ##### Key changed/added classes (or packages if there are too many classes) 
in this PR
   
   - `AbstractAuditLogger`
   - `TrustedChannelFailureHandler`
   - `TrustedChannelAuditServerEventHandler`
   - `TrustedChannelAuditHandshakeListener`
   - `ConsensusConfig`
   - ConfigNode/DataNode internal RPC client and service entry points
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to