Skip to content

[8.9] Fix publicUrlWarning message related to server.publicBaseUrl (#161091)#161110

Merged
kibanamachine merged 1 commit intoelastic:8.9from
kibanamachine:backport/8.9/pr-161091
Jul 3, 2023
Merged

[8.9] Fix publicUrlWarning message related to server.publicBaseUrl (#161091)#161110
kibanamachine merged 1 commit intoelastic:8.9from
kibanamachine:backport/8.9/pr-161091

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.9:

Questions ?

Please refer to the Backport tool documentation

…#161091)

Fixes elastic#160909

## Summary

This PR fixes the publicUrlWarning. Some of the URLs such as
`context.alertDetailsUrl` and `rule.url` are empty strings if
`server.publicBaseUrl` is not configured.

<img
src="https://github.com/elastic/kibana/assets/12370520/644e9a3e-8545-4b9b-84a9-1402815ac53d"
width=300 />

This is where you can see the message if `server.publicBaseUrl` is not
configured:

|Before|After|
|---|---|

|![image](https://github.com/elastic/kibana/assets/12370520/4d56d2fb-af2e-46f1-a3cf-91dd4cf9aaf2)|![image](https://github.com/elastic/kibana/assets/12370520/87c8bdb7-6d0b-4685-822e-c989f0013f42)|

(cherry picked from commit e9a9f03)
@kibanamachine kibanamachine merged commit ac1cf9e into elastic:8.9 Jul 3, 2023
@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

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
triggersActionsUi 1.4MB 1.4MB +17.0B
Unknown metric groups

ESLint disabled line counts

id before after diff
enterpriseSearch 13 15 +2
securitySolution 415 419 +4
total +6

Total ESLint disabled count

id before after diff
enterpriseSearch 14 16 +2
securitySolution 496 500 +4
total +6

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

cc @maryam-saeidi

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants