Merge pull request node-js-libs#49 from jhurliman/master

Add response.url parameter and give jsdom proper URLs
airhorns · Aug 18, 2011 · f39a4fb · f39a4fb
2 parents a942cf3 + 94b9426
commit f39a4fb
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/lib/node.io/dom.js b/lib/node.io/dom.js
@@ -52,7 +52,7 @@ Job.prototype.parseHtml = function (data, callback, response) {
         };
         var $, window, jquery, default_$;
         try {
-            window = require('jsdom').jsdom(data, null, {features:features}).createWindow();
+            window = require('jsdom').jsdom(data, null, {features:features, url:response.url}).createWindow();
             jquery = require('jquery');
             default_$ = jquery.create(window);
             $ = function (selector, context) {

diff --git a/lib/node.io/request.js b/lib/node.io/request.js
@@ -311,6 +311,8 @@ Job.prototype.doRequest = function (method, resource, body, headers, callback, p
 
     request = (secure ? https : http).request(options, function (response) {
 
+        response.url = resource;
+
         request_reponse = response;
 
         if (self.is_complete) {