[Entity Analytics][Privileged Monitoring]Fix index update failing due to invalid API payload#250758
Conversation
|
Pinging @elastic/security-entity-analytics (Team:Entity Analytics) |
hop-dev
left a comment
There was a problem hiding this comment.
Sorry this is definitely my bad, I added the extra API validation
|
Starting backport for target branches: 9.3 https://github.com/elastic/kibana/actions/runs/21477780397 |
@hop-dev Nothing to apologise for - easy to miss and no one spotted in the PR either. Ty for approval ^^ |
… to invalid API payload (elastic#250758) **Copilot:** This pull request makes a minor update to the API call in the `useEntityAnalyticsRoutes` hook. The change removes the unnecessary `type: 'index'` property from the request body, simplifying the payload. ### Summary This fixes a bug causing an issue where, after creating the initial index in the management page, no new indices could be selected, as the update payload no longer supports the type field. **Desk Testing Steps:** 1. Load up kibana 2. Add a couple of indicies from the manage sources page of privmon 3. Add a user or two to both indicies 4. Add one index to manage sources - index, update privileged users. 5. Try to delete that index from the options, and add the second instead. OR add the second index and try to update privileged users. 6. Should see - both / either indicies are available on update. **Desk Test Demo:** https://github.com/user-attachments/assets/1c4f5e87-9b03-4b75-92ca-450c5f5cfca3 (cherry picked from commit 412a84d)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
… to invalid API payload (elastic#250758) **Copilot:** This pull request makes a minor update to the API call in the `useEntityAnalyticsRoutes` hook. The change removes the unnecessary `type: 'index'` property from the request body, simplifying the payload. ### Summary This fixes a bug causing an issue where, after creating the initial index in the management page, no new indices could be selected, as the update payload no longer supports the type field. **Desk Testing Steps:** 1. Load up kibana 2. Add a couple of indicies from the manage sources page of privmon 3. Add a user or two to both indicies 4. Add one index to manage sources - index, update privileged users. 5. Try to delete that index from the options, and add the second instead. OR add the second index and try to update privileged users. 6. Should see - both / either indicies are available on update. **Desk Test Demo:** https://github.com/user-attachments/assets/1c4f5e87-9b03-4b75-92ca-450c5f5cfca3
|
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
3 similar comments
|
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
|
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
|
Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync. |
…ng due to invalid API payload (#250758) (#250885) # Backport This will backport the following commits from `main` to `9.3`: - [[Entity Analytics][Privileged Monitoring]Fix index update failing due to invalid API payload (#250758)](#250758) <!--- Backport version: 9.6.6 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"Charlotte Alexandra Wilson","email":"CAWilson94@users.noreply.github.com"},"sourceCommit":{"committedDate":"2026-01-29T12:16:02Z","message":"[Entity Analytics][Privileged Monitoring]Fix index update failing due to invalid API payload (#250758)\n\n**Copilot:** This pull request makes a minor update to the API call in\nthe `useEntityAnalyticsRoutes` hook. The change removes the unnecessary\n`type: 'index'` property from the request body, simplifying the payload.\n\n### Summary\nThis fixes a bug causing an issue where, after creating the initial\nindex in the management page, no new indices could be selected, as the\nupdate payload no longer supports the type field.\n\n**Desk Testing Steps:**\n\n1. Load up kibana\n2. Add a couple of indicies from the manage sources page of privmon\n3. Add a user or two to both indicies\n4. Add one index to manage sources - index, update privileged users.\n5. Try to delete that index from the options, and add the second\ninstead. OR add the second index and try to update privileged users.\n6. Should see - both / either indicies are available on update. \n\n**Desk Test Demo:** \n\n\nhttps://github.com/user-attachments/assets/1c4f5e87-9b03-4b75-92ca-450c5f5cfca3","sha":"412a84d1770299490a22e551272bac27dac2d635","branchLabelMapping":{"^v9.4.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","Team:Entity Analytics","backport:version","v9.3.0","v9.4.0"],"title":"[Entity Analytics][Privileged Monitoring]Fix index update failing due to invalid API payload","number":250758,"url":"https://github.com/elastic/kibana/pull/250758","mergeCommit":{"message":"[Entity Analytics][Privileged Monitoring]Fix index update failing due to invalid API payload (#250758)\n\n**Copilot:** This pull request makes a minor update to the API call in\nthe `useEntityAnalyticsRoutes` hook. The change removes the unnecessary\n`type: 'index'` property from the request body, simplifying the payload.\n\n### Summary\nThis fixes a bug causing an issue where, after creating the initial\nindex in the management page, no new indices could be selected, as the\nupdate payload no longer supports the type field.\n\n**Desk Testing Steps:**\n\n1. Load up kibana\n2. Add a couple of indicies from the manage sources page of privmon\n3. Add a user or two to both indicies\n4. Add one index to manage sources - index, update privileged users.\n5. Try to delete that index from the options, and add the second\ninstead. OR add the second index and try to update privileged users.\n6. Should see - both / either indicies are available on update. \n\n**Desk Test Demo:** \n\n\nhttps://github.com/user-attachments/assets/1c4f5e87-9b03-4b75-92ca-450c5f5cfca3","sha":"412a84d1770299490a22e551272bac27dac2d635"}},"sourceBranch":"main","suggestedTargetBranches":["9.3"],"targetPullRequestStates":[{"branch":"9.3","label":"v9.3.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.4.0","branchLabelMappingKey":"^v9.4.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/250758","number":250758,"mergeCommit":{"message":"[Entity Analytics][Privileged Monitoring]Fix index update failing due to invalid API payload (#250758)\n\n**Copilot:** This pull request makes a minor update to the API call in\nthe `useEntityAnalyticsRoutes` hook. The change removes the unnecessary\n`type: 'index'` property from the request body, simplifying the payload.\n\n### Summary\nThis fixes a bug causing an issue where, after creating the initial\nindex in the management page, no new indices could be selected, as the\nupdate payload no longer supports the type field.\n\n**Desk Testing Steps:**\n\n1. Load up kibana\n2. Add a couple of indicies from the manage sources page of privmon\n3. Add a user or two to both indicies\n4. Add one index to manage sources - index, update privileged users.\n5. Try to delete that index from the options, and add the second\ninstead. OR add the second index and try to update privileged users.\n6. Should see - both / either indicies are available on update. \n\n**Desk Test Demo:** \n\n\nhttps://github.com/user-attachments/assets/1c4f5e87-9b03-4b75-92ca-450c5f5cfca3","sha":"412a84d1770299490a22e551272bac27dac2d635"}}]}] BACKPORT--> Co-authored-by: Charlotte Alexandra Wilson <CAWilson94@users.noreply.github.com>
Copilot: This pull request makes a minor update to the API call in the
useEntityAnalyticsRouteshook. The change removes the unnecessarytype: 'index'property from the request body, simplifying the payload.Summary
This fixes a bug causing an issue where, after creating the initial index in the management page, no new indices could be selected, as the update payload no longer supports the type field.
Desk Testing Steps:
Desk Test Demo:
Screen.Recording.2026-01-28.at.16.18.05.mov