-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Lens] CSV export #74509
Comments
Pinging @elastic/kibana-app (Team:KibanaApp) |
Answered questions:
|
Proposal: To use the currently rendered data in the chart, we need to lift it out of the expression. This can be done using adapters. Adapters are meant to be a serializable data structure to make it possible to pass them between execution contexts. The current data adapter doesn't provide this, so we should implement a Lens specific solution for this for now and unify later with legacy adapters.
Open questions:
|
This is probably a good idea. There's a relevant discussion in #56081 around escaping output and/or warning about potentially dangerous output that could lead to CSV command injection. The reporting team implemented protections in #63645, but looking at #56081, it seems that the data table & inspector don't have similar controls in place yet. If |
Some discussed questions:
Yes, in a second step we can do this (and also have support for esdocs, essql, esdsl, ... out of the box)
Not sure, maybe? We would need more meta information on the table though. Peter will keep it in mind in alerting planning meetings
Clarify API (maybe enforce some hooks to make it clear how one should use it) |
Fixed by #83430 |
The current tables behind the rendered chart should be downloadable as CSV similar to the ingration in Visualize.
This includes a way to do so from the editor as well as from the context menu on Dashboards.
The biggest part of this can probably be solved by #55902
The text was updated successfully, but these errors were encountered: