diff options
| author | Luke Kanies <luke@madstop.com> | 2008-06-11 18:52:05 -0500 |
|---|---|---|
| committer | Luke Kanies <luke@madstop.com> | 2008-06-11 18:52:05 -0500 |
| commit | 4b29a5ea234e2c0c36ab5fca8fa0f6dee0085a7e (patch) | |
| tree | ea71d478f74bcb7cae8b12cabb14d9630f83b288 /lib/puppet | |
| parent | 041ca27460013088b790fab377aba0a8e470ce51 (diff) | |
| download | puppet-4b29a5ea234e2c0c36ab5fca8fa0f6dee0085a7e.tar.gz puppet-4b29a5ea234e2c0c36ab5fca8fa0f6dee0085a7e.tar.xz puppet-4b29a5ea234e2c0c36ab5fca8fa0f6dee0085a7e.zip | |
Fixing how the indirection tests for whether the request has node info.
My request authorization code changed the request to set its 'node'
instance variable, rather than using its options, and I apparently
didn't change the Indirection class to do this.
I have *no* idea how these tests were passing before -- they passed for
me yesterday but failed for Andrew, and today they started failing for
me. Frightening.
Diffstat (limited to 'lib/puppet')
| -rw-r--r-- | lib/puppet/indirector/indirection.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/puppet/indirector/indirection.rb b/lib/puppet/indirector/indirection.rb index 05464f8c9..4841ec532 100644 --- a/lib/puppet/indirector/indirection.rb +++ b/lib/puppet/indirector/indirection.rb @@ -267,7 +267,7 @@ class Puppet::Indirector::Indirection def check_authorization(request, terminus) # At this point, we're assuming authorization makes no sense without # client information. - return unless request.options[:node] + return unless request.node # This is only to authorize via a terminus-specific authorization hook. return unless terminus.respond_to?(:authorized?) |