Skip to content

[8.x] [Synthtrace] Fix wrong url build in the Kibana client (#217678)#217877

Merged
kibanamachine merged 3 commits intoelastic:8.19from
kibanamachine:backport/8.x/pr-217678
May 28, 2025
Merged

[8.x] [Synthtrace] Fix wrong url build in the Kibana client (#217678)#217877
kibanamachine merged 3 commits intoelastic:8.19from
kibanamachine:backport/8.x/pr-217678

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

Relates to elastic#217529
elastic#216653
elastic#216844

## Summary

The issue was introduced in the [PR
here](https://github.com/elastic/kibana/pull/212120/files#diff-34f8e7299930135fd708d98018fc6f4141d6e7c25df7e5fdb90f3472ad0e2948R36):
basically, the URL will look like: `
http:/user:pass@localhost:5620/api/fleet/epm/packages/apm?prerelease=false`
because `Path.join` will strip the `/` which is needed in this case -
this URL is also passed to `getFetchAgent`. This PR will fix this issue.

(cherry picked from commit 7f0a625)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Apr 10, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) April 10, 2025 15:56
@botelastic botelastic bot added the Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation. label Apr 10, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@jbudz jbudz changed the base branch from old-8.x to 8.19 April 18, 2025 18:41
@jbudz
Copy link
Contributor

jbudz commented Apr 18, 2025

@jennypavlova

I updated the target branch from old-8.x (previously 8.x) to 8.19 to reflect our global branch rename of 8.x to 8.19.

When you have a moment can you double check everything is in order and proceed as usual if so? Feel free to reach out if there's any questions.

@jennypavlova
Copy link
Member

I updated the target branch from old-8.x (previously 8.x) to 8.19 to reflect our global branch rename of 8.x to 8.19.
When you have a moment can you double check everything is in order and proceed as usual if so? Feel free to reach out if there's any questions.

@jbudz Thank you! LGTM :) I think this should be the only backport needed so it's good to go

@kibanamachine kibanamachine merged commit a93bfec into elastic:8.19 May 28, 2025
8 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #11 / Security Solution - Telemetry Security Telemetry - Indices metadata task telemetry @ess indices metadata should publish ilm policy events

Metrics [docs]

✅ unchanged

History

cc @jennypavlova

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 Team:obs-ux-infra_services - DEPRECATED DEPRECATED - Use Team:obs-presentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants