Skip to content
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

Region and coordinate maps can no longer zoom beyond level 10 with default distribution #62856

Closed
thomasneirynck opened this issue Apr 7, 2020 · 7 comments · Fixed by #62883
Assignees
Labels
bug Fixes for quality problems that affect the customer experience [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation regression Team:Visualizations Visualization editors, elastic-charts and infrastructure v7.7.0 v7.8.0 v8.0.0

Comments

@thomasneirynck
Copy link
Contributor

thomasneirynck commented Apr 7, 2020

Kibana version:

7.7, 7.x, Master

Describe the bug:

The license-key parameter is no longer added to request for EMS for the region and coordinate map visualizations. This causes the map not to zoom beyond level 10.

The default distribution of Kibana ships with the tilemap plugin (kibana/x-pack/legacy/plugins/tilemap).

This plugin adds the license-key to all CORS-requests to EMS. This ensures that the user has access to zoom level 18 in the region and coordinate map. It does so by calling a method on the ServiceSettings object.

serviceSettings.addQueryParams({ license: tileMapPluginInfo.license.uid });
serviceSettings.disableZoomMessage();

This code-path no longer executes. This causes the map to stop zooming at level 10 and showing a warning.

Steps to reproduce:

  1. Run default distribution of Kibana
  2. Add the following to kibana.yml:

xpack.maps.showMapVisualizationTypes: true

  1. Install sample data, go to Visualize,, open a new coordinate or region maps
  2. Zoom-in on the Map -> the Map will stop zooming at level 10

Expected behavior:

In default distribution, the region and coordinate maps shoudl zoom beyond level 10, to level 18.

Screenshots (if relevant):

image

Provide logs and/or server output (if relevant):

Note that the license parameter is not added to the URLs for EMS

e.g.: from browser console.

https://tiles.maps.elastic.co/styles/osm-bright-desaturated/10/263/392.png?elastic_tile_service_tos=agree&my_app_name=kibana&my_app_version=7.7.0

@thomasneirynck thomasneirynck added bug Fixes for quality problems that affect the customer experience regression [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation Team:Visualizations Visualization editors, elastic-charts and infrastructure v7.7.0 v7.8.0 v8.0.0 labels Apr 7, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-gis (Team:Geo)

@elasticmachine
Copy link
Contributor

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

@thomasneirynck
Copy link
Contributor Author

cc @aaronjcaldwell @joshdover @ppisljar @timroes

It seems like that plugin is the only one that still uses vis_type_enhancers, which I presume got removed. This module (

uiRoutes.addSetupWork(function($injector, serviceSettings) {
) is not getting executed.

Perhaps any of you know more about this?

@timroes
Copy link
Contributor

timroes commented Apr 7, 2020

Cc @flash1293

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app-arch (Team:AppArch)

@flash1293
Copy link
Contributor

flash1293 commented Apr 8, 2020

@thomasneirynck @timroes So I checked the code and I'm not 100% sure how ui exports work. But when I interpret

https://github.com/elastic/kibana/blob/master/src/legacy/ui/ui_exports/ui_export_types/ui_app_extensions.js#L62

correctly, then visTypeEnhancers gets wrapped up into visTypes.

The main kibana app bundle stopped pulling in these ui-exports in:

#58337 (c81e0da#diff-da8fc2674a5c5002e66fd47cd70158a4L29)

From what I can tell this is the only place this happened.

@thomasneirynck
Copy link
Contributor Author

thx @flash1293 for the added context. @aaronjcaldwell proposes to replace this by using hacks #62883, which works as expected.

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 [Deprecated-Use Team:Presentation]Team:Geo Former Team Label for Geo Team. Now use Team:Presentation regression Team:Visualizations Visualization editors, elastic-charts and infrastructure v7.7.0 v7.8.0 v8.0.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants