Skip to content

[ti_eset] Fix grok processor not consuming spaces in IoC pattern - #19126

Closed
polakovicp wants to merge 2 commits into
elastic:ti_set_polakovicp_enhancementsfrom
polakovicp:ti_eset_domains_fix
Closed

polakovicp wants to merge 2 commits into
elastic:ti_set_polakovicp_enhancementsfrom
polakovicp:ti_eset_domains_fix

Conversation

@polakovicp

Copy link
Copy Markdown
Contributor

[ti_eset] Fix grok processor not consuming spaces in IoC pattern

PR fixes IoC pattern parsing issue in domains feed in ti_eset integration. Documentation for this data stream is submitted as a separate PR

@polakovicp
polakovicp requested a review from a team as a code owner May 21, 2026 14:46
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Reviewers

Buildkite won't run for external contributors automatically; you need to add a comment:

  • /test : will kick off a build in Buildkite.

NOTE: https://github.com/elastic/integrations/blob/main/.buildkite/pull-requests.json contains all those details.

@andrewkroh andrewkroh added Integration:ti_eset ESET Threat Intelligence (Partner supported) Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] labels May 21, 2026
@infra-vault-gh-plugin-prod

Copy link
Copy Markdown

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

@botelastic

botelastic Bot commented Jun 20, 2026

Copy link
Copy Markdown

Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!

@botelastic botelastic Bot added the Stalled label Jun 20, 2026
@mergify

mergify Bot commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@vera-review-bot

Copy link
Copy Markdown

👀 I have started reviewing the PR

@vera-review-bot

Copy link
Copy Markdown

Vera Review Bot

For the current commit state, I did not find any issues.


🤖 AI-Generated Review | Vera Review Bot | 📚 Knowledge base: integration-skills

⚠️ Automated review — verify suggestions before applying.

@efd6 efd6 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@polakovicp Where is the documentation addition?

This will need to have a changelog entry (bug-fix), and a version bump (patch level).

@botelastic botelastic Bot removed the Stalled label Jun 22, 2026
@vera-review-bot

Copy link
Copy Markdown

⚠️ @polakovicp I couldn't complete the review for commits 57bbfd4 after several attempts. A maintainer can re-run it with @vera-review-bot review.

@botelastic

botelastic Bot commented Aug 16, 2026

Copy link
Copy Markdown

Hi! We just realized that we haven't looked into this PR in a while. We're sorry! We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1. Thank you for your contribution!

@botelastic botelastic Bot added the Stalled label Aug 16, 2026
@botelastic

botelastic Bot commented Sep 15, 2026

Copy link
Copy Markdown

Hi! This PR has been stale for a while and we're going to close it as part of our cleanup procedure. We appreciate your contribution and would like to apologize if we have not been able to review it, due to the current heavy load of the team. Feel free to re-open this PR if you think it should stay open and is worth rebasing. Thank you for your contribution!

@botelastic botelastic Bot closed this Sep 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Integration:ti_eset ESET Threat Intelligence (Partner supported) Stalled Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants