Skip to content

[UI Metrics] Support multi-colon keys#76913

Merged
afharo merged 1 commit intoelastic:masterfrom
afharo:ui-metrics/support-multicolon-keys
Sep 9, 2020
Merged

[UI Metrics] Support multi-colon keys#76913
afharo merged 1 commit intoelastic:masterfrom
afharo:ui-metrics/support-multicolon-keys

Conversation

@afharo
Copy link
Copy Markdown
Member

@afharo afharo commented Sep 8, 2020

Summary

Adds support for multi-colon keys in UI Metrics. This is needed for the kibana-user_agent to work as expected.

The current behaviour, from an user agent like Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:80.0) Gecko/20100101 Firefox/80.0, it only reports Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv (it drops the text after the first :).

Checklist

Delete any items that are not applicable to this PR.

For maintainers

@afharo afharo added bug Fixes for quality problems that affect the customer experience Feature:Telemetry release_note:skip Skip the PR/issue when compiling release notes v7.10.0 v7.9.2 labels Sep 8, 2020
@afharo afharo requested a review from a team as a code owner September 8, 2020 09:41
@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

Copy link
Copy Markdown
Contributor

@TinaHeiligers TinaHeiligers left a comment

Choose a reason for hiding this comment

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

LGTM

@afharo afharo merged commit 61edcb3 into elastic:master Sep 9, 2020
@afharo afharo deleted the ui-metrics/support-multicolon-keys branch September 9, 2020 10:38
afharo added a commit to afharo/kibana that referenced this pull request Sep 9, 2020
@afharo afharo mentioned this pull request Sep 9, 2020
7 tasks
gmmorris added a commit to gmmorris/kibana that referenced this pull request Sep 9, 2020
* master: (68 commits)
  a11y tests on spaces home page including feature control  (elastic#76515)
  [ML] Transforms list: persist pagination through refresh interval (elastic#76786)
  [ML] Replace all use of date_histogram interval with fixed_interval (elastic#76876)
  [Timelion] Update timelion deprecation links (elastic#77008)
  [Security Solution] Refactor Network Details to use Search Strategy (elastic#76928)
  Upgrade elastic charts to 21.1.2 (elastic#76939)
  [Alerting][Connectors] Refactor Jira: Generic Implementation (phase one) (elastic#73778)
  [Snapshot & Restore] fix pre existing policy with no existing repository (elastic#76861)
  Update saved object management UI text (elastic#76826)
  [Form lib] Add validations prop to UseArray and expose "moveItem" handler (elastic#76949)
  [Logs UI] Use fields api in log stream (elastic#76919)
  [UI Metrics] Support multi-colon keys (elastic#76913)
  [APM] Script for creating functional test archive (elastic#76926)
  [ENDPOINT] First version of the trusted apps list. (elastic#76304)
  Correct field for rum page url (elastic#76916)
  [Security Solution] Fix redirect properly old SIEM App routes (elastic#76868)
  Bump http-proxy from 1.17.0 to 1.18.1 (elastic#76924)
  [RUM Dashboard] Visitor breakdown usability (elastic#76834)
  [Search] Add a new advanced setting searchTimeout (elastic#75728)
  [DOCS] Adds timelion deprecation to new visualize docs structure (elastic#76959)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Fixes for quality problems that affect the customer experience Feature:Telemetry release_note:skip Skip the PR/issue when compiling release notes v7.9.2 v7.10.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants