masquerade 0.6.0 → 0.7.0

data/CHANGELOG.rdoc

@@ -1,5 +1,9 @@
 = Changelog
 
+== Relase 0.7.0 - May 03, 2011
+
+* FIX: Catching an invalid user or group error also catching the block's ArgumentErrors
+
 == Release 0.6.0 - April 15, 2011
 
 * CHANGE: Change both uid+euid and gid+egid to switch user

data/Manifest

@@ -3,4 +3,3 @@ Manifest
 README.rdoc
 Rakefile
 lib/masquerade.rb
-masquerade.gemspec

data/Rakefile

@@ -2,7 +2,7 @@ require "rubygems"
 require "rake"
 require "echoe"
 
-Echoe.new("masquerade", "0.6.0") do |p|
+Echoe.new("masquerade", "0.7.0") do |p|
   p.description = "A user/group impersonator. Allows you to run a block of code as another user or group."
   p.summary = "A user/group impersonator. Allows you to run a block of code as another user or group"
   p.url = "http://rubygems.org/gems/masquerade"

data/lib/masquerade.rb

@@ -47,11 +47,11 @@ Returns the result of the block call. So you can do:
 =end
 module Masquerade
   def self.as(who)
+    current_euid = Process.euid
+    current_egid = Process.egid
+    current_uid = Process.uid
+    current_gid = Process.gid
     begin
-      current_euid = Process.euid
-      current_egid = Process.egid
-      current_uid = Process.uid
-      current_gid = Process.gid
       if who.include? :group
         group_struct = nil
         if who[:group].kind_of? String
@@ -78,19 +78,19 @@ module Masquerade
         Process.euid = user_struct.uid
         Process.uid = user_struct.uid
       end
-      # run the block
-      response = yield who
-      # restore process uid and gid
-      Process.uid = current_uid
-      Process.euid = current_euid
-      Process.gid = current_gid
-      Process.egid = current_egid
-      return response
     rescue Errno::EPERM
       raise PermissionsError, "You do not have permissions to impersonate this user or group"
     rescue ArgumentError
       raise BadUserOrGroupError, "The user or group does not exist"
     end
+    # run the block
+    response = yield who
+    # restore process uid and gid
+    Process.uid = current_uid
+    Process.euid = current_euid
+    Process.gid = current_gid
+    Process.egid = current_egid
+    return response
   end
   
   class PermissionsError < StandardError

data/masquerade.gemspec

@@ -2,11 +2,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{masquerade}
-  s.version = "0.6.0"
+  s.version = "0.7.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 1.2") if s.respond_to? :required_rubygems_version=
   s.authors = ["Nitesh Goel"]
-  s.date = %q{2011-04-16}
+  s.date = %q{2011-05-03}
   s.description = %q{A user/group impersonator. Allows you to run a block of code as another user or group.}
   s.email = %q{nitesh@wikinvest.com}
   s.extra_rdoc_files = ["CHANGELOG.rdoc", "README.rdoc", "lib/masquerade.rb"]

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: masquerade
 version: !ruby/object:Gem::Version 
-  hash: 7
+  hash: 3
   prerelease: 
   segments: 
   - 0
-  - 6
+  - 7
   - 0
-  version: 0.6.0
+  version: 0.7.0
 platform: ruby
 authors: 
 - Nitesh Goel
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-04-16 00:00:00 Z
+date: 2011-05-03 00:00:00 Z
 dependencies: []
 
 description: A user/group impersonator. Allows you to run a block of code as another user or group.