Skip to content

Conversation

@idovandijk
Copy link
Contributor

Status

Ready

Description

The task that validates IPs handled cases where the IP field doesn't exist, and where the IP is malformed. Fixed an edge case where null IPs specifically were not handled, which resulted in the RegexExtractAll tranformer to fail.

Must have

  • Tests
  • Documentation

@content-bot
Copy link
Collaborator

This PR was automatically updated by a GitHub Action

  • CortexResponseAndRemediation pack version was bumped to 1.1.94.

To stop automatic version bumps, add the ignore-auto-bump-version label to the github PR.

@idovandijk idovandijk added the ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines label Jul 23, 2025
@idovandijk idovandijk changed the title A used executed multiple LDAP enumeration queries - IP validation task fix A user executed multiple LDAP enumeration queries - IP validation task fix Jul 23, 2025
@content-bot
Copy link
Collaborator

Validate summary
The following errors were thrown as a part of this pr: .
If the AG100 validation in the pre-commit GitHub Action fails, the pull request cannot be force-merged.

Verdict: PR can be force merged from validate perspective? ✅

@idovandijk idovandijk merged commit 8e9d6ee into master Jul 24, 2025
17 checks passed
@idovandijk idovandijk deleted the a-user-executed-multiple-ldap-enumeration-queries-client-validation-fix branch July 24, 2025 13:45
xsoar-bot pushed a commit to xsoar-contrib/content that referenced this pull request Sep 17, 2025
…k fix (demisto#40689)

* pb, format, rn

* Bump pack from version CortexResponseAndRemediation to 1.1.94.

* added continueonerror for core-get-endpoints

---------

Co-authored-by: Content Bot <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-approved ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants