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

[8.10] [Security Solution] Fix missing hash in sync to url (#166847) #167010

Merged
merged 5 commits into from
Sep 29, 2023
Merged

[8.10] [Security Solution] Fix missing hash in sync to url (#166847) #167010

merged 5 commits into from
Sep 29, 2023

Conversation

kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.10:

Questions ?

Please refer to the Backport tool documentation

@lgestc
[Security Solution] Fix missing hash in sync to url (elastic#166847)
e35b279 
## Summary

This PR fixes the root cause for
elastic#166686 and
elastic#166774

@angorayc @machadoum

### Checklist

Delete any items that are not applicable to this PR.

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

(cherry picked from commit d52c5a1)
@kibanamachine kibanamachine enabled auto-merge (squash) September 22, 2023 08:30
@kibanamachine kibanamachine mentioned this pull request Sep 22, 2023
Merged
1 task
@MadameSheema
Copy link
Member

@lgestc can you please check this backport? Thanks!

@PhilippeOberti
Copy link
Contributor

@elasticmachine merge upstream

@PhilippeOberti
Copy link
Contributor

@elasticmachine merge upstream

@PhilippeOberti
Copy link
Contributor

@elasticmachine merge upstream

@PhilippeOberti
Copy link
Contributor

@elasticmachine merge upstream

@kibana-ci
Copy link
Collaborator

kibana-ci commented Sep 29, 2023 •
edited
Loading

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] Defend Workflows Cypress Tests #1 / Artifact tabs in Policy Details page Trusted applications tab "before all" hook for "[NONE] User cannot see the tab for Trusted applications" "before all" hook for "[NONE] User cannot see the tab for Trusted applications"
  • [job] [logs] Defend Workflows Cypress Tests #1 / Isolate command from Alerts "before all" hook for "should isolate and release host" "before all" hook for "should isolate and release host"
  • [job] [logs] Defend Workflows Cypress Tests #1 / Isolate command from Cases "before all" hook for "should isolate and release host" "before all" hook for "should isolate and release host"
  • [job] [logs] Defend Workflows Cypress Tests #1 / Isolate command from Manage "before all" hook for "should allow filtering endpoint by Isolated status" "before all" hook for "should allow filtering endpoint by Isolated status"
  • [job] [logs] Security Solution Cypress Tests #4 / Open timeline Open timeline modal "before each" hook for "should display timeline info - description" "before each" hook for "should display timeline info - description"

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 15.6MB 15.6MB +375.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @lgestc

@kibanamachine kibanamachine merged commit 657eaab into elastic:8.10 Sep 29, 2023
1 check passed
This was referenced Oct 1, 2023
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@github-actions github-actions[bot] github-actions[bot] approved these changes

Assignees

@lgestc lgestc

Labels
backport
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@kibanamachine @MadameSheema @PhilippeOberti @kibana-ci @lgestc