Skip to content

Conversation

@ruturajsumo
Copy link
Collaborator

Purpose of this pull request

This pull request The original code assumed --fieldAllRecords would always be a single string. However, when multiple values were passed, it caused unexpected behavior. To handle this, the logic was updated to loop through each field name (assuming a list or space-separated string) during record processing, ensuring values from multiple fields could be extracted properly.

Select the type of change

  • Minor Changes - Typos, formatting, slight revisions
  • Update Content - Revisions, updating sections
  • New Content - New features, sections, pages, tutorials
  • Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

https://sumologic.atlassian.net/browse/CSOAR-3521

@cla-bot cla-bot bot added the cla-signed Contributor approved, listed in .clabot file label Jul 3, 2025
@ruturajsumo ruturajsumo marked this pull request as draft July 3, 2025 11:35
@ruturajsumo ruturajsumo self-assigned this Jul 3, 2025
@ruturajsumo ruturajsumo marked this pull request as ready for review July 4, 2025 12:07
@ruturajsumo
Copy link
Collaborator Author

@jpipkin1 Please merge this PR

Copy link
Collaborator

@kimsauce kimsauce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@kimsauce kimsauce enabled auto-merge July 4, 2025 18:19
@kimsauce kimsauce added this pull request to the merge queue Jul 4, 2025
Merged via the queue into SumoLogic:main with commit e751b0e Jul 4, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed Contributor approved, listed in .clabot file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants