finish issues with JSON

README.md

@@ -1,9 +1,8 @@
 <h1 align="center">ipmap</h1>
 
-<p align="center">
-An interactive map that shows connected IP addresses.
-<img src=data/screenshot.png>
-</p>
+<p align="center">An interactive map that shows connected IP addresses.</p>
+<hr>
+<p align="center"><img src=data/screenshot.png></p>
 
 ## Requirements 
 ipmap uses `libpcap-dev`, which is only easily available on UNIX-like systems (Linux, MacOS, *BSD).
@@ -49,8 +48,8 @@ If no service specified, ipapi will be used, which will limit how many IP is det
 
 ## Command Line Options
 ```
-ipmap 0.1.2
-Skyline High School Coding Club Authors <[email protected]>
+ipmap 0.1.5
+Skyline High Coding Club Authors
 
 USAGE:
     ipmap [FLAGS] [OPTIONS]
@@ -61,5 +60,7 @@ FLAGS:
     -V, --version     Prints version information
 
 OPTIONS:
-    -s, --service <SERVICE>    Geolocation API
+    -p, --port <PORT>             Set webserver port, default port 700
+    -s, --service <SERVICE>       Choose Geolocation API [possible values: ipwhois, ipapi, ipapico, freegeoip]
+    -w, --write-to-file <PATH>    Set path to write JSON to
 ```

data/map.js

@@ -8,7 +8,7 @@ var alreadyAdded = new Set();
 
 function addMarkers(jsonText) {
 	console.log(jsonText)
-	for(var i = 1; i < jsonText.length; i++) {
+	for(var i = 0; i < jsonText.length; i++) {
 		var obj = jsonText[i];
 
 		if (alreadyAdded.has(obj.ip)) {
@@ -17,7 +17,7 @@ function addMarkers(jsonText) {
 
 		console.log(obj.ip);
 
-		document.getElementById("totalIps").innerHTML = jsonText.length - 1 + " Unique IP's";
+		document.getElementById("totalIps").innerHTML = jsonText.length + " Unique IP's";
 
 		L.marker([obj.latitude, obj.longitude]).addTo(ipmap)
 // .bindpopup(obj.ip);

src/ip.rs

@@ -265,7 +265,11 @@ pub fn get_document() -> String {
 
     json.push_str("[\n");
 
-    for a in &*IP_MAP.read().unwrap() {
+    let v = &*IP_MAP.read().unwrap();
+
+    let iter = v[1..].iter();
+
+    for a in iter {
         let address = IPAddress {
             ip: a[0].to_owned(),
             latitude: a[1].to_owned(),

src/main.rs

@@ -59,7 +59,7 @@ fn main() {
             Arg::with_name("port")
                 .long("port")
                 .short("p")
-                .help("Set webserver port, default port 700")
+                .help("Set webserver port, if not set it defaults to port 700")
                 .required(false)
                 .takes_value(true)
                 .value_name("PORT"),

test

@@ -0,0 +1,3 @@
+[
+{"ip":"193.148.18.75","latitude":"40.7157","longitude":"-74"}
+]