Skip to content

[Core] Remove URL Overflow & Deprecate storeInSessionStorage setting#242972

Merged
ThomThomson merged 19 commits intoelastic:mainfrom
ThomThomson:deprecateHashUrlSetting
Jan 30, 2026
Merged

[Core] Remove URL Overflow & Deprecate storeInSessionStorage setting#242972
ThomThomson merged 19 commits intoelastic:mainfrom
ThomThomson:deprecateHashUrlSetting

Conversation

@ThomThomson
Copy link
Copy Markdown
Contributor

@ThomThomson ThomThomson commented Nov 13, 2025

URLs without limits

This PR removes the artificial URL length limit from Kibana. This was added when IE was part of the support matrix of Kibana, and when chromium based browsers had a hard URL length limit. This limit no longer exists anymore in our supported browsers, so it's high-time to remove our artificial limit.

Deprecation

This PR also deprecates the advanced setting which was advertised to work around our URL length limit. This advanced setting has been causing problems with Discover and Dashboards for quite some time, as it prevented the ability to copy paste URLs directly between tabs.

Follow up to #242788

@ThomThomson
Copy link
Copy Markdown
Contributor Author

/ci

@ThomThomson ThomThomson added release_note:deprecation loe:small Small Level of Effort impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. backport:skip This PR does not require backporting labels Nov 14, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Nov 14, 2025

🔍 Preview links for changed docs

@ThomThomson
Copy link
Copy Markdown
Contributor Author

/ci

@ThomThomson
Copy link
Copy Markdown
Contributor Author

/ci

@ThomThomson
Copy link
Copy Markdown
Contributor Author

/ci

@ThomThomson
Copy link
Copy Markdown
Contributor Author

/ci

@ThomThomson
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@ThomThomson
Copy link
Copy Markdown
Contributor Author

/ci

@ThomThomson ThomThomson marked this pull request as ready for review January 20, 2026 19:43
@ThomThomson ThomThomson requested review from a team as code owners January 20, 2026 19:43
@ThomThomson ThomThomson requested review from a team as code owners January 20, 2026 19:43
Co-authored-by: shainaraskas <58563081+shainaraskas@users.noreply.github.com>
Copy link
Copy Markdown
Member

@florent-leborgne florent-leborgne left a comment

Choose a reason for hiding this comment

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

LGTM for docs (I pushed the quick change suggested by Shaina to the applies_to tagging)

@ThomThomson
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

Copy link
Copy Markdown
Contributor

@nreese nreese left a comment

Choose a reason for hiding this comment

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

kibana-presentation changes LGTM
code review only

@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #7 / discover/tabs navigation should go back to the last active tab after returning from Surrounding Docs page

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
core 546 544 -2

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
aiAssistantManagementSelection 91.4KB 91.5KB +92.0B
maps 3.1MB 3.1MB -23.0B
total +69.0B

Page load bundle

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

id before after diff
core 496.6KB 493.9KB -2.7KB
Unknown metric groups

API count

id before after diff
@kbn/core-apps-browser-internal 24 23 -1

History

@ThomThomson ThomThomson merged commit 00c1852 into elastic:main Jan 30, 2026
16 checks passed
hannahbrooks pushed a commit to hannahbrooks/kibana that referenced this pull request Jan 30, 2026
mbondyra added a commit to mbondyra/kibana that referenced this pull request Feb 2, 2026
…iew_cps

* commit '32efd9b2fb078ade51073fd2d0068bc74c029d6b': (49 commits)
  [Security Solution] Rules exceptions subfeatures (elastic#245722)
  [BK] Upgrade axios (elastic#251150)
  Fix AI Connector form fields resetting to default value when cleared by user (elastic#251095)
  deduplicate otel dependencies (elastic#250841)
  Adds initial agents.md file (elastic#250833)
  [index management] Faster index list loading (elastic#246276)
  skip failing test suite (elastic#251086)
  skip failing test suite (elastic#251048)
  [Security Solutions] Trial Companion - adjust UX design (elastic#250910)
  [Traces][Discover] Prevent flyout remount when switching document types in Trace Waterfall (elastic#250406)
  [DOCS][Cases][9.4 & Serverless]: Doc new `Maximum amount of cases to open` setting for case action (elastic#250993)
  [Discover][Traces] Explore trace.id from logs in Discover (elastic#249632)
  Remove ! from SOs docs link (elastic#251097)
  [ML] Maps: Add telemetry events for file uploads (elastic#247543)
  [Fleet] Fix dupplicate ids when copying an integration policy or an agent policy (elastic#250971)
  [Dashboards as Code] Add snake case object keys util (elastic#250962)
  [Core] Remove URL Overflow & Deprecate `storeInSessionStorage` setting (elastic#242972)
  [One Workflow] fix: Fix Variable Retrieval in Workflow Execution Engine (elastic#250852)
  Rework Elastic Managed LLMs page (elastic#251069)
  [Lens powered by ES|QL] Update Switch to Query mode modal warning message (elastic#251051)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. loe:small Small Level of Effort release_note:deprecation v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants