-
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
Fix 302 on /favicon.ico #137294
Fix 302 on /favicon.ico #137294
Conversation
Pinging @elastic/kibana-core (Team:Core) |
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.
Seems reasonable to me
💚 Build Succeeded
Metrics [docs]
History
To update your PR or re-run it, just comment with: |
(cherry picked from commit a40d241)
(cherry picked from commit a40d241)
💔 Some backports could not be created
Note: Successful backport PRs will be merged automatically after passing CI. Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
(cherry picked from commit a40d241) Co-authored-by: Gerard Soldevila <[email protected]>
(cherry picked from commit a40d241) Co-authored-by: Gerard Soldevila <[email protected]> Co-authored-by: Spencer <[email protected]>
Fixes #31683
@elastic/kibana-operations do you see any drawback in simply adding the missing favicon.ico, so that we avoid the whole conflict?