Skip to content

[Pipe] Add receiver session authentication integration tests - #18395

Open
Caideyipi wants to merge 1 commit into
masterfrom
test/pipe-receiver-session-it
Open

[Pipe] Add receiver session authentication integration tests#18395
Caideyipi wants to merge 1 commit into
masterfrom
test/pipe-receiver-session-it

Conversation

@Caideyipi

Copy link
Copy Markdown
Collaborator

Description

Add integration-test coverage for the pipe receiver authentication and session changes in #17741, #18111, and commit e32465e2 (#18084).

Legacy pipe receiver

  • Verify legacy handshake, file transfer, and pipe-data RPCs require login and USE_PIPE.
  • Verify deletion and TsFile loading use the authenticated user's permissions.
  • Verify handshake and TsFile loading do not auto-create databases or schema when auto creation is disabled.

DataNode and ConfigNode receivers

  • Verify V1 handshakes are rejected and V2 handshakes validate credentials.
  • Verify unauthenticated requests are rejected while authenticated client sessions remain usable.
  • Verify successful DataNode receiver handshakes establish a session that can transfer tablets.

Two-stage aggregation

  • Verify count-point-processor uses the source username and password for its internal two-stage aggregate sender.
  • Run the source and result sink in separate 1C1D clusters and assert the combined count is transferred successfully.

Verification

  • mvn clean package -DskipTests -Drat.skip=true -pl integration-test -am -P with-integration-tests
  • IoTDBLegacyPipeReceiverAutoCreateIT
  • All added IoTDBLegacyPipeReceiverSecurityIT methods
  • IoTDBPipeReceiverSessionIT
  • IoTDBPipeTwoStageAggregateIT#testCountPointProcessorUsesSourceCredentials
  • Spotless, Checkstyle, and git diff --check

This PR has:

  • been self-reviewed.
  • added integration tests.
  • been tested in test IoTDB clusters.

Key changed/added classes
  • IoTDBLegacyPipeReceiverSecurityIT
  • IoTDBLegacyPipeReceiverAutoCreateIT
  • IoTDBPipeReceiverSessionIT
  • IoTDBPipeTwoStageAggregateIT

@codecov

codecov Bot commented Aug 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 43.49%. Comparing base (f6a844d) to head (9eab84f).

Additional details and impacted files
@@            Coverage Diff            @@
##             master   #18395   +/-   ##
=========================================
  Coverage     43.49%   43.49%           
  Complexity      374      374           
=========================================
  Files          5394     5394           
  Lines        385391   385391           
  Branches      50148    50148           
=========================================
+ Hits         167624   167626    +2     
+ Misses       217767   217765    -2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant