Fix #2424 - File server can't find module in environment

Actually, the issue is:
*  when the web server gets the request, it creates an indirection
request, filling attributes like ip or node from the HTTP request.
To do this, all the interesting attributes are given in a hash
(called options, see P::I::Request#new).
Once the request is properly initialized the options hash doesn't
contain the ip or node information (see set_attributes)

* the request is then transmitted to the file_serving layer,
which happily wants to use the node attribute to find environments or
perform authorization.
Unfortunately it fetches the node value from the request options hash,
not the request itself.
Since this node information is empty, puppet fails to find the
proper mount point, and fails the download.

This change makes sure we pass all the way down the node and fix
the authorization check.

Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>

2 files changed, 4 insertions, 4 deletions

diff --git a/spec/unit/file_serving/configuration.rb b/spec/unit/file_serving/configuration.rb
index 60515bf1e..9545f01cc 100755
--- a/spec/unit/file_serving/configuration.rb
+++ b/spec/unit/file_serving/configuration.rb
@@ -150,7 +150,7 @@ describe Puppet::FileServing::Configuration do
             @config = Puppet::FileServing::Configuration.create
             @config.stubs(:find_mount)
 
-            @request = stub 'request', :key => "foo/bar/baz", :options => {}
+            @request = stub 'request', :key => "foo/bar/baz", :options => {}, :node => nil
         end
 
         it "should reread the configuration" do
@@ -179,7 +179,7 @@ describe Puppet::FileServing::Configuration do
 
         it "should use the mount name and node to find the mount" do
             @config.expects(:find_mount).with { |name, node| name == "foo" and node == "mynode" }
-            @request.options[:node] = "mynode"
+            @request.stubs(:node).returns("mynode")
 
             @config.split_path(@request)
         end
diff --git a/spec/unit/indirector/file_server.rb b/spec/unit/indirector/file_server.rb
index a80f5aed3..7dab320c1 100755
--- a/spec/unit/indirector/file_server.rb
+++ b/spec/unit/indirector/file_server.rb
@@ -241,8 +241,8 @@ describe Puppet::Indirector::FileServer do
             @mount = stub 'mount'
             @configuration.expects(:split_path).with(@request).returns([@mount, "rel/path"])
 
-            @request.options[:node] = "mynode"
-            @request.options[:ipaddress] = "myip"
+            @request.stubs(:node).returns("mynode")
+            @request.stubs(:ip).returns("myip")
             @mount.expects(:allowed?).with("mynode", "myip").returns "something"
 
             @file_server.authorized?(@request).should == "something"