From: Tom Hughes <tom@compton.nu>
Date: Mon, 10 Feb 2020 15:26:00 +0000 (+0000)
Subject: Fix generation of JSON data for tile server map
X-Git-Url: https://git.openstreetmap.org/dns.git/commitdiff_plain/c56a2f4216218c5d0c1cb78206aed695b2a4b2e1

Fix generation of JSON data for tile server map
---

diff --git a/bin/mkgeo b/bin/mkgeo
index d942ee0..77b44d9 100755
--- a/bin/mkgeo
+++ b/bin/mkgeo
@@ -212,48 +212,51 @@ my $jsonfile = IO::File->new("> json/${zone}.json") || die "$!";
 # Output headers
 $jszonefile->print("var \U${jszone}\E_RECORDS = [\n");
 
-# Skip outputting server map if gdnsd is doing that
-if (!defined($gdnsname))
+# Output details for each country
+foreach my $origin (sort { $a->{name} cmp $b->{name} } values %$origins)
 {
-    # Output details for each country
-    foreach my $origin (sort { $a->{name} cmp $b->{name} } values %$origins)
-    {
-        my $cluster = $origin->{cluster};
-        my $clon = $origin->{lon};
-        my $clat = $origin->{lat};
-        my $slon = $cluster->{lon};
-        my $slat = $cluster->{lat};
+    my $cluster = $origin->{cluster};
+    my $clon = $origin->{lon};
+    my $clat = $origin->{lat};
+    my $slon = $cluster->{lon};
+    my $slat = $cluster->{lat};
 
-        if ($clon > 0 && $slon < 0 && 360 + $slon - $clon < $clon - $slon)
-        {
-            $slon = $slon + 360;
-        }
-        elsif ($slon > 0 && $clon < 0 && 360 + $clon - $slon < $slon - $clon)
-        {
-            $clon = $clon + 360;
-        }
+    if ($clon > 0 && $slon < 0 && 360 + $slon - $clon < $clon - $slon)
+    {
+        $slon = $slon + 360;
+    }
+    elsif ($slon > 0 && $clon < 0 && 360 + $clon - $slon < $slon - $clon)
+    {
+        $clon = $clon + 360;
+    }
 
+    if (!defined($gdnsname))
+    {
         $zonefile->print("# $origin->{name}\n");
         $zonefile->print("C\L$origin->{code}\E.${zone}:$cluster->{name}.${zone}:600\n");
 
         $jszonefile->print("  CNAME(\"\L$origin->{code}\E.${jszone}\", \"$cluster->{name}.${zone}.\", TTL(\"10m\")),\n");
+    }
 
-        push @json, {
-            type => "Feature",
-            geometry => {
-                type => "LineString",
-                coordinates => [ [ $clon, $clat ], [ $slon, $slat ] ]
-            },
-            properties => {
-                origin => $origin->{name},
-                server => $cluster->{name},
-                colour => $cluster->{colour}
-            }
-        };
+    push @json, {
+        type => "Feature",
+        geometry => {
+            type => "LineString",
+            coordinates => [ [ $clon, $clat ], [ $slon, $slat ] ]
+        },
+        properties => {
+            origin => $origin->{name},
+            server => $cluster->{name},
+            colour => $cluster->{colour}
+        }
+    };
 
-        $targetorigins->{$cluster->{name}}->{bandwidth} += $origin->{bandwidth};
-    }
+    $targetorigins->{$cluster->{name}}->{bandwidth} += $origin->{bandwidth};
+}
 
+# Skip default records if we don't need them
+if (!defined($gdnsname))
+{
     # Header for default records
     $zonefile->print("# Unknown origins\n");