Skip to content
  • Peter Wu's avatar
    Re-implement "Map" feature for Endpoints · ba1ef0ec
    Peter Wu authored
    This feature was removed in v2.5.1rc0-427-gf529ab5d0a, anticipating that
    MaxMind would remove support for it in 2019. They have however changed
    their mind and maintained latitude and longitude information.
    
    They recommend displaying an accuracy radius, but the reported values
    are 50, 100, 200 and 1000km. When implemented literally, a marker in
    Ireland would cover the whole island plus mainland, so I have instead
    opted to use a fixed radius of 1km at deeper zoom levels.
    
    The old ipmap.html file was outdated and had broken tiles, I rewrote a
    new one from scratch using the light-weight Leaflet library combined
    with tiles from OpenStreetMap. This is more mobile-friendly and secure
    (https, SRI). To improve handling of nearby or overlapping nodes,
    clustering is used (individual nodes can still be inspected).
    
    Browser compatibility results: IE8 is unusable, IE9 partially works
    (tooltips sometimes disappear and the cluster radius control is gone),
    IE11 works. Of course Firefox 65 and Chromium 72 have no issues.
    
    The map popup description in the generated GeoJSON structure is now
    split in several properties, allowing presentation to be handled by the
    HTML page instead of the C code.
    
    Bug: 14693
    Change-Id: If2ec9c518f7723ac0ab27b6272463356875a0ff2
    Reviewed-on: https://code.wireshark.org/review/31952
    
    
    Petri-Dish: Peter Wu <peter@lekensteyn.nl>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: default avatarGerald Combs <gerald@wireshark.org>
    Reviewed-by: default avatarPeter Wu <peter@lekensteyn.nl>
    ba1ef0ec