-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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.
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
[Security Solution] ensure fleetfilenotfound is reported as a 404 #164738
Conversation
Pinging @elastic/security-defend-workflows (Team:Defend Workflows) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀 Needs a test for this too. We can add it in a new PR as well
💚 Build Succeeded
Metrics [docs]
To update your PR or re-run it, just comment with: |
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…astic#164738) (cherry picked from commit 9e89a96)
…404 (#164738) (#164836) # Backport This will backport the following commits from `main` to `8.10`: - [[Security Solution] ensure fleetfilenotfound is reported as a 404 (#164738)](#164738) <!--- Backport version: 8.9.8 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Dan Panzarella","email":"[email protected]"},"sourceCommit":{"committedDate":"2023-08-24T19:48:10Z","message":"[Security Solution] ensure fleetfilenotfound is reported as a 404 (#164738)","sha":"9e89a9694b9bacc275f8d2125200b53447a13f1f","branchLabelMapping":{"^v8.11.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Defend Workflows","v8.10.0","v8.11.0"],"number":164738,"url":"https://github.com/elastic/kibana/pull/164738","mergeCommit":{"message":"[Security Solution] ensure fleetfilenotfound is reported as a 404 (#164738)","sha":"9e89a9694b9bacc275f8d2125200b53447a13f1f"}},"sourceBranch":"main","suggestedTargetBranches":["8.10"],"targetPullRequestStates":[{"branch":"8.10","label":"v8.10.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.11.0","labelRegex":"^v8.11.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/164738","number":164738,"mergeCommit":{"message":"[Security Solution] ensure fleetfilenotfound is reported as a 404 (#164738)","sha":"9e89a9694b9bacc275f8d2125200b53447a13f1f"}}]}] BACKPORT--> Co-authored-by: Dan Panzarella <[email protected]> Co-authored-by: Kibana Machine <[email protected]>
Summary
This fixes a bug during upgrade, wherein files uploaded prior to version 8.10 are deleted, but the error messaging displayed a 500 error. This has been fixed to show a proper message
For maintainers