Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 21, 2025

Backport of #24221 to 6.2 branch.

When a field value is removed from an Aggregation Event Definition condition in the UI, the frontend was sending an empty string instead of null. The backend expects Optional.empty (null) when no field should be used, causing search failures.

Changes:

  • Set config.series.field to null when the field value is removed in the UI
  • Prevents empty string from being sent to backend for optional field parameters

Fixes #24007


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

ryan-carroll-graylog and others added 5 commits November 17, 2025 22:17
…ions (#24221)

* convert cleared field values to null so backend handles them correctly

* add cl entry and test

* Backend fix

---------

Co-authored-by: Zack King <[email protected]>
(cherry picked from commit 55fce40)
* Initial plan

* Initial plan for fixing linter errors

Co-authored-by: ryan-carroll-graylog <[email protected]>

* Fix linter errors in NumberRefExpression.test.jsx

Co-authored-by: ryan-carroll-graylog <[email protected]>

* Delete graylog2-web-interface/package-lock.json

* Revert yarn.lock to original state

Co-authored-by: ryan-carroll-graylog <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: ryan-carroll-graylog <[email protected]>
Co-authored-by: Ryan Carroll <[email protected]>
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copilot AI changed the title [WIP] Clear removed 'Field' values from Aggregation Event Definition conditions Clear removed 'Field' values from Aggregation Event Definition conditions (6.2) Nov 21, 2025
Base automatically changed from backport-6.2/24007-clear-the-field-specified-in-event-definition-aggregation-breaks-the-event-definition-can-never-be-matched-as-having-results to 6.2 November 24, 2025 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants