Skip to content

[dev/1.3] Pipe: Harden legacy pipe file transfer validation and access checks#17791

Open
Caideyipi wants to merge 2 commits into
dev/1.3from
cp-33c3ef7-dev-1.3
Open

[dev/1.3] Pipe: Harden legacy pipe file transfer validation and access checks#17791
Caideyipi wants to merge 2 commits into
dev/1.3from
cp-33c3ef7-dev-1.3

Conversation

@Caideyipi
Copy link
Copy Markdown
Collaborator

Backport/cherry-pick of 33c3ef7 (#17741) to dev/1.3. Testing: .\mvnw.cmd -pl iotdb-core/datanode spotless:apply passed. Targeted IoTDBLegacyPipeReceiverAgentTest could not complete locally because datanode compile fails on existing pipe API mismatch outside this change.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 1, 2026

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 1, 2026

Codecov Report

❌ Patch coverage is 27.41935% with 45 lines in your changes missing coverage. Please review.
✅ Project coverage is 42.39%. Comparing base (5d4277a) to head (258f7c1).

Files with missing lines Patch % Lines
.../db/protocol/thrift/impl/ClientRPCServiceImpl.java 0.00% 24 Missing ⚠️
...pipe/sink/protocol/legacy/IoTDBLegacyPipeSink.java 0.00% 16 Missing ⚠️
.../protocol/legacy/IoTDBLegacyPipeReceiverAgent.java 77.27% 5 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             dev/1.3   #17791      +/-   ##
=============================================
+ Coverage      42.33%   42.39%   +0.05%     
  Complexity       241      241              
=============================================
  Files           3616     3616              
  Lines         239350   239396      +46     
  Branches       29098    29104       +6     
=============================================
+ Hits          101324   101485     +161     
+ Misses        138026   137911     -115     

☔ View full report in Codecov by Sentry.
📢 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