Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Security Solution][8.0.0-SNAPSHOT]Rule navigation breaking for Event Rendered View Alert list #120711

Closed
ghost opened this issue Dec 8, 2021 · 11 comments · Fixed by #121139
Closed

[Security Solution][8.0.0-SNAPSHOT]Rule navigation breaking for Event Rendered View Alert list #120711

ghost opened this issue Dec 8, 2021 · 11 comments · Fixed by #121139
Assignees
@kqualters-elastic
Labels
bug Fixes for quality problems that affect the customer experience fixed impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:Threat Hunting:Investigations Security Solution Investigations Team v8.0.0

Comments

@ghost
Copy link

ghost commented Dec 8, 2021

Describe the bug
Rule navigation breaking for Event Rendered View Alert list

Build Details

Version: 8.0.0-SNAPSHOT
Commit:ad3660f3acbfe6eb809d869b908221edf2846313
Build:48594

Steps

  • generate some detection alert and Custom alert
  • Change alert list view to event rendered view
  • click on rule name to navigate to rule detail page
  • observed that navigation breaks and application not found came up

image

Whats working
Rule navigation is been working on below area

  • Grid view
    • view details : Document summary
    • view rule details page from cell actions
  • Event Rendered view
    • view details : Document summary

Screen-cast

navigation.mp4

Additional Details:

Kibana URL : https://bc7-b2385e.kb.europe-west1.gcp.cloud.es.io:9243
Navigation Link for Event Rendered view ( breaking URL 🔴 ) :
https://snapshot-6f2d37.kb.us-central1.gcp.qa.cld.elstc.co:9243/app/securitySolution/id/8ee78710-57e8-11ec-a2bf-7b160ac6f95b?sourcerer=(default:(id:security-solution-default,selectedPatterns:!(%27auditbeat-*%27,%27logs-*%27)))&timerange=(global:(linkTo:!(timeline),timerange:(from:%272021-12-07T18:30:00.000Z%27,fromStr:now%2Fd,kind:relative,to:%272021-12-08T18:29:59.999Z%27,toStr:now%2Fd)),timeline:(linkTo:!(global),timerange:(from:%272021-12-07T18:30:00.000Z%27,fromStr:now%2Fd,kind:relative,to:%272021-12-08T18:29:59.999Z%27,toStr:now%2Fd)))

Same Rule Navigation Link for Grid view ( Working URL 🟢 ) :

https://snapshot-6f2d37.kb.us-central1.gcp.qa.cld.elstc.co:9243/app/security/rules/id/8ee78710-57e8-11ec-a2bf-7b160ac6f95b?sourcerer=(default:(id:security-solution-default,selectedPatterns:!(%27auditbeat-*%27,%27logs-*%27)))&timerange=(global:(linkTo:!(timeline),timerange:(from:%272021-12-07T18:30:00.000Z%27,fromStr:now%2Fd,kind:relative,to:%272021-12-08T18:29:59.999Z%27,toStr:now%2Fd)),timeline:(linkTo:!(global),timerange:(from:%272021-12-07T18:30:00.000Z%27,fromStr:now%2Fd,kind:relative,to:%272021-12-08T18:29:59.999Z%27,toStr:now%2Fd)))

image
@ghost ghost added bug Fixes for quality problems that affect the customer experience triage_needed Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. labels Dec 8, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-solution (Team: SecuritySolution)

@ghost ghost added impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. v8.0.0 labels Dec 8, 2021
@manishgupta-qasource
Copy link

Reviewed & assigned to @MadameSheema

@ghost ghost added high and removed impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. labels Dec 8, 2021
@michaelolo24 michaelolo24 added Team:Threat Hunting:Investigations Security Solution Investigations Team and removed triage_needed labels Dec 8, 2021
@michaelolo24
Copy link
Contributor

Thanks for finding this @karanbirsingh-qasource ! Can you please help us verify this was not happening on 7.16 as well?

@kqualters-elastic
Copy link
Contributor

https://github.com/elastic/kibana/blob/main/x-pack/plugins/timelines/public/components/rule_name/index.tsx#L27 and https://github.com/elastic/kibana/blob/main/x-pack/plugins/timelines/public/components/rule_name/index.tsx#L36 need to be 'securitySolutionUI'

@kqualters-elastic
Copy link
Contributor

@michaelolo24 will not be a problem in any 7.x branches, as APP_UI_ID is not a concept, only APP_ID https://github.com/elastic/kibana/blob/7.16/x-pack/plugins/security_solution/common/constants.ts#L12

@ghost
Copy link
Author

ghost commented Dec 9, 2021

Thanks for finding this @karanbirsingh-qasource ! Can you please help us verify this was not happening on 7.16 as well?

@michaelolo24 Issue is not occuring on 7.16.0 BC7

rule-navigation.mp4

@kqualters-elastic kqualters-elastic mentioned this issue Dec 9, 2021
Closed
@MadameSheema MadameSheema added impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. and removed high labels Dec 9, 2021
@kqualters-elastic kqualters-elastic mentioned this issue Dec 13, 2021
Merged
1 task
@ghost
Copy link
Author

ghost commented Dec 22, 2021

Hi @michaelolo24

we have validated this issue on 8.0.0-SNAPSHOT and found it Fixed 🟢 .

Build Details:

Version: 8.0.0-SNAPSHOT
commit:002f9fae38acdf71d6df88d808a742976de22cc8
Build:48805

Snap-Shoot:

Rule_Navigation.mp4

Hence we are closing this issue.

thanks!!

@ghost ghost closed this as completed Dec 22, 2021
@ghost
Copy link
Author

ghost commented Jan 20, 2022

Hi @MadameSheema

Validated this issue on 8.0.0-RC2 and its still fixed here too 🟢

Build Details:

Version:8.0.0-RC-2
Commit:93d9bd54452711c449fe07e4f23c43a4a0e519c3
Build:49092

Screen cast

120711.mp4

@ghost
Copy link
Author

ghost commented Feb 4, 2022

Hi @MadameSheema

Regressed the issue on 8.0.0-GA BC1 and its still fixed here too. ✔️

Build Details:

Version 8.0.0-GA
Commit 57ca5e139a33dd2eed927ce98d8231a1f217cd15
Build 49192

Screen-Cast

event-render.mp4

@ghost
Copy link
Author

ghost commented Feb 11, 2022

Hi @MadameSheema

we have validated this issue under re-bug regress part on 8.1.0-BC2 and found this issue still to be fixed here 🟢 .

Please find below build details:

Build Details:

Version: 8.1.0-BC2
Commit:ee89ebfddeda3baaf6cd87c0299247c5248cb952
Build:50222

Screen-Cast:

120711.mp4

thanks !!

@ghost
Copy link
Author

ghost commented Feb 22, 2022

Hi @MadameSheema

we have validated this issue under re-bug regress part on 8.0.1-BC1 and found this issue still to be fixed here 🟢 .

Please find below build details:

Build Details:

Version: 8.1.0-BC2
Commit:ee89ebfddeda3baaf6cd87c0299247c5248cb952
Build:50222

Screen-Cast:

event-render-view.mp4

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kqualters-elastic kqualters-elastic

Labels
bug Fixes for quality problems that affect the customer experience fixed impact:high Addressing this issue will have a high level of impact on the quality/strength of our product. Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:Threat Hunting:Investigations Security Solution Investigations Team v8.0.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Security Solution] Add app id prop for navigateToApp kqualters-elastic/kibana
5 participants
@elasticmachine @michaelolo24 @MadameSheema @kqualters-elastic @manishgupta-qasource