Skip to content

fix: POSIX line-continuation in process_command splitting#105

Merged
yndu13 merged 1 commit into
masterfrom
feature/aone-84424188-line-continuation
Jul 18, 2026
Merged

fix: POSIX line-continuation in process_command splitting#105
yndu13 merged 1 commit into
masterfrom
feature/aone-84424188-line-continuation

Conversation

@yndu13

@yndu13 yndu13 commented Jul 18, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Treat backslash-newline as POSIX line continuation (both removed) in CommandLineUtils.split, outside quotes and inside double quotes, matching shlex semantics (follow-up to fix: quote-aware External process_command parsing #104).
  • Add test cases for both continuation positions.

@codecov-commenter

codecov-commenter commented Jul 18, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 85.71429% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 87.22%. Comparing base (b3a2335) to head (ae5e25c).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
...com/aliyun/credentials/utils/CommandLineUtils.java 85.71% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #105      +/-   ##
============================================
+ Coverage     87.20%   87.22%   +0.02%     
- Complexity      838      840       +2     
============================================
  Files            53       53              
  Lines          3102     3108       +6     
  Branches        306      308       +2     
============================================
+ Hits           2705     2711       +6     
  Misses          289      289              
  Partials        108      108              

☔ 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.

@yndu13
yndu13 merged commit 0341a31 into master Jul 18, 2026
10 checks passed
@yndu13
yndu13 deleted the feature/aone-84424188-line-continuation branch July 18, 2026 04:19
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.

2 participants