Skip to content

Commit

Permalink
Remove dependency on parseUri library
Browse files Browse the repository at this point in the history
Resolves issues around there being two parseUri and parseuri npm modules in the registry. Newer versions of npm do not like this and there will be more issues from this in the future once npm v3 is released, so the dependency was removed.
  • Loading branch information
peter-murray authored and peter-murray committed Sep 17, 2015
1 parent 886ef40 commit 93ca0b3
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 6 deletions.
8 changes: 4 additions & 4 deletions hue-api/bridge-discovery.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
"use strict";

var Q = require("q")
, parseUri = require("parseUri")
var url = require("url")
, Q = require("q")
, search = require("./search")
, http = require("./httpPromise")
, utils = require("./utils")
Expand Down Expand Up @@ -157,10 +157,10 @@ function _getHueBridgeHost(description) {
var uri;

if (_isHueBridge(description)) {
uri = parseUri(description.url);
uri = url.parse(description.url);
return {
"id": description.model.serial,
"ipaddress": uri.host
"ipaddress": uri.hostname
};
}
return null;
Expand Down
3 changes: 1 addition & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@
],
"main": "index.js",
"scripts": {
"test": "make test"
"test": "mocha test"
},
"repository": {
"type": "git",
Expand All @@ -40,7 +40,6 @@
"xml2js": "~0.2.2",
"q": "~1.0.0",
"request-util": "~0.1.0",
"parseUri": "~1.2.3-2",
"traits": "~0.4.0"
},
"devDependencies": {
Expand Down

0 comments on commit 93ca0b3

Please sign in to comment.