[BUG] SSH multiRow parsing shifts columns in aligned or tab-separated output
Maintainers usually reply within 1 day
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 75/100
Research direction
Start with the multiRow parser and SshCollectImplTest, which the issue identifies through its debug logs. Run the tests with the supplied aligned and tab-separated stdout examples and inspect how the parser splits header and data lines. Done means aliases map to the expected name, size, and used values while preserving missing-column handling.
Written by the indexing model from the issue text.
Description
Is there an existing issue for this?
- I searched existing issues and pull requests.
Current Behavior
The multiRow parser splits on a literal single space. Padding creates empty tokens and tabs stay inside tokens, so field aliases can receive the wrong value or NULL for normal aligned command output.
Expected Behavior
Trim each header/data line and split runs of whitespace. Keep alias-based projection and missing-column handling.
Steps To Reproduce
Use parseType=multiRow, aliases name,size,used, and stdout " NAME\t SIZE USED \ndisk1\t40G 15G \n disk2 20G\t5G" (\t/\n denote tabs/newlines). Expected rows disk1,40G,15G and disk2,20G,5G. The current parser misaligns those columns.
Environment
master at 40f5aac47e332f5240cc1a58ba7d201b4f456d2a. macOS, JDK 26 targeting Java 25.
Debug logs
JDK 26 targeting Java 25: SshCollectImplTest, 18 tests passed using Maven verify with default checkstyle/enforcer. 2 tests failed against the original parser. Coverage reporting was skipped because no JaCoCo execution file was produced.
- Dominant language
- Java
- Stars
- 7.4k
- Forks
- 1.3k
- Avg merge
- 2d 12h
- Merged PRs (30d)
- 34
Getting set up
Starts the project's dev container in your browser, under your own GitHub account.
- No Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from apache/hertzbeat
-
[Feature] <title>When selecting monitoring items, the "Select All on Current Page" checkbox will only display issuesPossibly taken @butterfly-520-boy claimed this 3 days ago. Opennew feature
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
apache/hertzbeat#4425 · 3 comments ·
Maintainers usually reply within 1 day
-
[Feature] <title>The "Select All" option is at the bottom, while more operations are above, which is quite inconvenient. Move the "Select All" box to the top to enhance the user experiencePossibly taken @butterfly-520-boy claimed this 3 days ago. Opennew feature
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
apache/hertzbeat#4340 · 1 comment ·
Maintainers usually reply within 1 day
-
question
Difficulty 5/5 Over a week Newbie friendliness 15/100
Maintainers usually reply within 1 day
-
[Feature] <title>Can the menu operation checkbox always hover at the top of the current page along with the scroll bar on the rightPossibly taken @butterfly-520-boy claimed this today. Opennew feature
Difficulty 3/5 1-2 days Newbie friendliness 48/100
apache/hertzbeat#4431 · 2 comments ·
Maintainers usually reply within 1 day
All issues in apache/hertzbeat
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
sqlcipher/sqlcipher-android#97 · 1 comment ·
-
bug IIIF interoperability
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Netcracker/qubership-integration-platform#1046 ·
Maintainers usually reply within 2 days
-
`check_java_version()` fails when Java path contains spaces (Windows / Git Bash, `C:\Program Files`)Open
Difficulty 2/5 1-3 hours Newbie friendliness 68/100