Skip to content

Fixes #69639: Ignore url.url fields above 2048 characters#69863

Merged
rudolf merged 1 commit intoelastic:masterfrom
rudolf:url-field-ignore-above-limit
Jun 25, 2020
Merged

Fixes #69639: Ignore url.url fields above 2048 characters#69863
rudolf merged 1 commit intoelastic:masterfrom
rudolf:url-field-ignore-above-limit

Conversation

@rudolf
Copy link
Copy Markdown
Contributor

@rudolf rudolf commented Jun 24, 2020

Summary

Fixes #69639 which causes Kibana upgrades (saved object migrations) to fail in v7.8.0 if they contain url saved objects which shortened a very large url (> 8191 characters). Migrations fail with the following error:
FATAL Error: Document contains at least one immense term in field="url.url.keyword" (whose UTF8 encoding is longer than the max length 32766)

Checklist

Delete any items that are not applicable to this PR.

For maintainers

@rudolf rudolf added Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v7.8.0 v7.8.1 labels Jun 24, 2020
@rudolf rudolf requested a review from a team as a code owner June 24, 2020 22:48
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-app (Team:KibanaApp)

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Build metrics

✅ unchanged

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

@rudolf rudolf removed the v7.8.0 label Jun 25, 2020
@rudolf rudolf merged commit 7a55782 into elastic:master Jun 25, 2020
@rudolf rudolf deleted the url-field-ignore-above-limit branch June 25, 2020 13:45
rudolf added a commit to rudolf/kibana that referenced this pull request Jun 25, 2020
rudolf added a commit to rudolf/kibana that referenced this pull request Jun 25, 2020
gmmorris added a commit to gmmorris/kibana that referenced this pull request Jun 25, 2020
* master: (45 commits)
  [QA] Unskip functional tests (elastic#69760)
  [SIEM][Detection Engine] - Update DE to work with new exceptions schema (elastic#69715)
  Fixes elastic#69639: Ignore url.url fields above 2048 characters (elastic#69863)
  PR: Provide limit warnings to user when API limits are reached. (elastic#69590)
  [Maps] Remove broken button (elastic#69853)
  Makes usage collection methods available on start (elastic#69836)
  [SIEM][CASE] Improve Jira's labelling (elastic#69892)
  [Logs UI] Access ML via the programmatic plugin API (elastic#68905)
  [ML] DF Analytics: Creation wizard part 3 (elastic#69456)
  Update Resolver generator script documentation (elastic#69912)
  [ML] Changes View results button text on new job page (elastic#69809)
  Add master branch to backport config (elastic#69893)
  [Ingest Manager] Kibana, not EPR, controls removable packages (elastic#69761)
  unskips 'Events columns' test (elastic#69684)
  [ML] Changes the ML overview empty analytics panel text (elastic#69801)
  [DOCS] Emphasizes where File Data Visualizer is located. (elastic#69812)
  add the `exactRoute` property to app registration (elastic#69772)
  Bump backport to 5.4.6 (elastic#69880)
  [Logs UI] ML log integration splash screen (elastic#69288)
  Clean up TSVB type client code to conform to the schema (elastic#68519)
  ...
gmmorris added a commit to gmmorris/kibana that referenced this pull request Jun 25, 2020
* master: (90 commits)
  [Encrypted Saved Objects] Adds support for migrations in ESO (elastic#69513)
  [SIEM] Replace WithSource with useWithSource hook (elastic#68722)
  [Endpoint]EMT-451: add ability to filter endpoint metadata based on presence of unenrolled events (elastic#69708)
  rename old siem kibana config to securitySolution (elastic#69874)
  Remove unused Resolver code (elastic#69914)
  [Observability] Fixing dynamic return type based on the appName (elastic#69894)
  [SECURITY SOLUTION][INGEST] Task/endpoint list tests (elastic#69419)
  Fixes special clicks and 3rd party icon sizes in nav (elastic#69767)
  [APM] Catch annotations index permission error and log warning (elastic#69881)
  [Endpoint][Ingest Manager] minor code cleanup (elastic#69844)
  [Logs UI] Logs ui context menu (elastic#69915)
  Index pattern serialize and de-serialize (elastic#68844)
  [QA] Unskip functional tests (elastic#69760)
  [SIEM][Detection Engine] - Update DE to work with new exceptions schema (elastic#69715)
  Fixes elastic#69639: Ignore url.url fields above 2048 characters (elastic#69863)
  PR: Provide limit warnings to user when API limits are reached. (elastic#69590)
  [Maps] Remove broken button (elastic#69853)
  Makes usage collection methods available on start (elastic#69836)
  [SIEM][CASE] Improve Jira's labelling (elastic#69892)
  [Logs UI] Access ML via the programmatic plugin API (elastic#68905)
  ...
rudolf added a commit that referenced this pull request Jun 26, 2020
…) (#69931)

* Fixes #69639: Ignore url.url fields above 2048 characters (#69863)

* Fix types in 7.8

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

Labels

release_note:fix Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v7.8.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migration from Kibana 7.7.1 to 7.8 fails with : Document contains at least one immense term ... whose UTF8 encoding is longer than the max length 32766

5 participants