Skip to content

[8.19] [Discover] Show ES|QL request URL in Inpector flyout (#221816)#222004

Merged
jughosta merged 1 commit intoelastic:8.19from
jughosta:backport/8.19/pr-221816
May 30, 2025
Merged

[8.19] [Discover] Show ES|QL request URL in Inpector flyout (#221816)#222004
jughosta merged 1 commit intoelastic:8.19from
jughosta:backport/8.19/pr-221816

Conversation

@jughosta
Copy link
Contributor

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

- Closes elastic#221581

## Summary

This PR adds the missing `requestParams` to the search interceptor code.
This makes the request URL appear again in Inpector flyout (Request
tab).

<img width="1072" alt="Screenshot 2025-05-28 at 17 55 09"
src="https://github.com/user-attachments/assets/1c11c7d7-1e49-4265-a014-88f369dad2a7"
/>

### Checklist

- [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
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

(cherry picked from commit 7fae9c5)

# Conflicts:
#	src/platform/plugins/shared/data/public/search/search_interceptor/search_interceptor.ts
@jughosta jughosta requested a review from kibanamachine as a code owner May 30, 2025 08:19
@jughosta jughosta added the backport This PR is a backport of another PR label May 30, 2025
@jughosta jughosta enabled auto-merge (squash) May 30, 2025 08:19
rawResponse: esqlResponse,
isPartial: esqlResponse.is_partial,
isRunning: esqlResponse.is_running,
requestParams,
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the case when bfetch is disabled via Advanced Settings.

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
data 401.7KB 401.7KB +16.0B

@jughosta jughosta requested a review from a team May 30, 2025 10:28
@jughosta jughosta merged commit 9d5c8eb into elastic:8.19 May 30, 2025
11 checks passed
@jughosta jughosta deleted the backport/8.19/pr-221816 branch May 30, 2025 10:52
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