fix: parse comma-separated list/stdin input targets#945
fix: parse comma-separated list/stdin input targets#945Tianlin0725 wants to merge 1 commit intoprojectdiscovery:devfrom
Conversation
Neo - PR Security ReviewCaution Review could not be completed Review could not be completed. Please retry with Suggestion: Try again with Comment |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@neo review Please re-run the security review. The previous attempt could not be completed. |
Proposed Changes
Fix input normalization so
-l/--listfile input and stdin handle comma-separated targets on a single line the same way as-u.processCommaSeparatedInputto split by comma, trim whitespace, and skip empty tokens.Test_ProcessCommaSeparatedInput.Proof
Repro behavior (before)
A single list line like:
was treated as one host entry and failed with:
Test evidence (after)
The new regression test verifies comma-separated entries are split into independent targets.
/claim #859