Merge pull request #4 from akoenig/feature/http-load

Functionality for parsing external files

README.md

@@ -24,6 +24,11 @@ gpxParse.parseGpx("<gpx></gpx>", function(error, data) {
 	//do stuff
 });
 
+// or an external file via HTTP(S)
+gpxParse.parseRemoteGpxFile("http://host.tld/my.gpx", function(error, data) {
+    //do stuff
+});
+
 ```
 
 #Tests

index.js

@@ -1 +1 @@
-module.exports = process.env.GPXPARSE_COV ? require('./lib-cov/gpx-parse') : require('./lib/gpx-parse');
+module.exports = require('./lib/gpx-parse');

lib/gpx-parse.js

@@ -2,7 +2,11 @@
 * @module gpx-parse
 **/
 
-var fs = require("fs"),
+var assert = require("assert"),
+    fs = require("fs"),
+    url = require("url"),
+    http = require("http"),
+    https = require("https"),
 	GpxResult = require('./gpxResult'),
 	GpxExtent = require('./gpxExtent'),
 	GpxWaypoint = require('./gpxWaypoint'),
@@ -176,6 +180,38 @@ exports.parseGpxFromFile = function(gpxFile, callback) {
 	});
 };
 
+/**
+ * Fetch a remote GPX file via HTTP(S) and parse it.
+ * 
+ * @param {string} uri The URI of the remote gpx file.
+ * @param {function} callback Invoked as `callback(err, data)`
+ *
+ */
+exports.parseRemoteGpxFile = function parseRemoteGpxFile (uri, callback) {
+    assert.equal(typeof uri, 'string', 'uri should be a string');
+    assert.equal(typeof callback, 'function', 'callback should be a function');
+
+    function onError (err) {
+        return callback(new Error('failed to fetch gpx file: ' + err.toString()));
+    }
+
+    function onResponse (res) {
+        var body = '';
+
+        res.on('data', function (data) {
+            body = body + data;
+        });
+
+        res.once('end', function () {
+            exports.parseGpx(body, callback);
+        });
+    }
+
+    (('https:' === url.parse(uri).protocol) ? https : http)
+        .get(uri, onResponse)
+        .once('error', onError);
+};
+
 //expose objects
 exports.GpxResult = GpxResult;
 exports.GpxExtent = GpxExtent;

package.json

@@ -10,19 +10,21 @@
     "geo"
   ],
   "dependencies": {
-    "xml2js": "*"
+    "xml2js": "^0.4.4"
   },
   "repository": {
     "type": "git",
     "url": "https://github.com/elliotstokes/gpx-parse.git"
   },
   "devDependencies": {
-    "nodeunit": "*",
-    "sandboxed-module": "*",
-    "jscoverage": "*",
     "coveralls": "*",
     "grunt": "^0.4.5",
-    "grunt-jsdoc": "^0.5.4"
+    "grunt-jsdoc": "^0.5.4",
+    "jscoverage": "*",
+    "nodeunit": "*",
+    "sandboxed-module": "*",
+    "server-destroy": "^1.0.0",
+    "st": "^0.5.1"
   },
   "scripts": {
     "test": "node_modules/.bin/nodeunit tests/",

tests/gpx-http.js

@@ -0,0 +1,38 @@
+var st = require('st')
+var http = require('http')
+
+var enableDestroy = require('server-destroy');
+
+var gpxParse = require("../");
+
+var server;
+
+module.exports = {
+
+    setUp: function(callback) {
+        server = http.createServer(
+            st(__dirname + '/integration')
+        ).listen(9090, function () {
+            enableDestroy(server);
+
+            callback();
+        });
+    },
+    tearDown: function(callback) {
+        // clean up
+        server.destroy();
+
+        callback();
+    },
+
+    "Test that remote gpx file can be loaded": function(test) {
+        gpxParse.parseRemoteGpxFile('http://localhost:9090/data/route-1.1.gpx', function (err, data) {
+            test.ok(data.metadata);
+            test.ok(data.waypoints);
+            test.ok(data.routes);
+            test.ok(data.tracks);
+
+            test.done();
+        });
+    }
+};