egoist 0.5.1 → 0.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ea4e36f6d1468e4ef6315c36155873a1f0a55889517a7642da29f67c61305d12
-  data.tar.gz: cb2e309cfe453ed80a05c0409c203b584efff5f2a13aac47a83d662831f6cc8d
+  metadata.gz: b6bd4a8956152dd8b62d79b7649ef0068c0ec77c8fa4f4a32fbfe6d045ecdc00
+  data.tar.gz: cc0ba2348f4802a38e5a8443b49f199f1e270155ba25bb92f61aef12db160619
 SHA512:
-  metadata.gz: 80b16f41f289bfe0de44e976838d0a7c94e134d573bfb1636ccb89438e8808794dec5c33ebdc70d60e7fe109198edd17021e04f516457e9957a9bc7c85dede15
-  data.tar.gz: b87b9ad5c35b61006c76d6fb4f520ae7a8e1ae9ca8e3f555f4c06a24cca72aea7f0cdce09bf4340b0de0abe6e5eedfabc7c4b09d04b806e15d6b48be62936531
+  metadata.gz: 2309402638c0d1be1aef9308102a8188d0fc70c20fc342ea6a2bc66208845766032a26e8de3ce8c00fc6da93dfda01dbad4e5784e87cb726677558a17c010efc
+  data.tar.gz: d2f54296ac9f21bd77710857e8e1ae5871708a177fa5b17ac5e9b9f906971631dcff856d470a629a967e918a577ff02b908e2e8b7e92f2607c51f739003c81ea

data/.version

@@ -1 +1 @@
-0.5.1
+0.9.0

data/lib/adapters/controller.rb

@@ -1,42 +1,13 @@
-klass =
-if defined? Rails
-  ActiveController::Base
-elsif defined? Lux
-  Lux::Controller
-end
-
-if klass
-  klass.class_eval do
-    def authorize *args, &block
-      opts = {}
-
-      @_is_policy_authorized = true
-
-      raise ArgumentErorr, 'authorize argument[s] not provided' unless args[0]
-
-      # authorize true
-      return if args[0].is_a? TrueClass
+# include Policy::Controller
 
-      if !args[1]
-        # authorize :admin?
-        opts[:action] = args.first
-      elsif args[2]
-        # authorize @model, write?, CustomClass
-        # authorize @model, write?, class: CustomClass
-        opts[:model]  = args.first
-        opts[:action] = args[1]
-        opts[:class]  = args[2].is_a?(Hash) ? args[2][:class] : args[2]
+class Policy
+  module Controller
+    def authorize result = false
+      if (block_given? ? yield : result)
+        @_is_policy_authorized = true
       else
-        # authorize @model, write?
-        opts[:model]  = args.first
-        opts[:action] = args[1]
+        Policy.error('Authorize did not pass truthy value')
       end
-
-      # covert all authorize actions to bang actions (fail unless true)
-      action = opts.delete(:action).to_s.sub('?', '!')
-
-      # do it
-      Policy(opts).send(action, &block)
     end
 
     def is_authorized?
@@ -44,8 +15,11 @@ if klass
     end
 
     def is_authorized!
-      raise ::Policy::Error.new('Request is not authorized!') unless is_authorized?
-      true
+      if is_authorized?
+        true
+      else
+        Policy.error('Request is not authorized!')
+      end
     end
   end
 end

data/lib/adapters/model.rb

@@ -1,21 +1,9 @@
-class Policy
-  module ModelAdapter
-    def self.can user, model
-      klass = '%sPolicy' % model.class
-      klass = Object.const_defined?(klass) ? klass.constantize : ::ModelPolicy
-      Policy(model: model || self, user: user, class: klass)
-    end
-  end
-end
+# include Policy::Model
 
-if defined? Rails
-  ActiveModel::Base.include Policy::ModelAdapter
-elsif defined? Sequel
-  class Sequel::Model
-    module InstanceMethods
-      def can user=nil
-        Policy::ModelAdapter.can user, self
-      end
+class Policy
+  module Model
+    def can user = nil
+      Policy.can model: self, user: user
     end
   end
 end

data/lib/egoist/base.rb

@@ -1,21 +1,9 @@
 class Policy
-  class << self
-    def can(model=nil, user=nil)
-      if model.is_a?(Hash)
-        user, model = model[:user], model[:model]
-      end
-
-      new(user: user, model: model).can
-    end
-  end
-
-  ###
-
   attr_reader :model, :user, :action
 
   def initialize model:, user: nil
     @model = model
-    @user  = user || current_user
+    @user  = user || Policy.current_user
   end
 
   # pass block if you want to handle errors yourself
@@ -28,8 +16,13 @@ class Policy
       .to_sym
 
     # pre check
-    raise RuntimeError, 'Method name not allowed' if %i(can).index(@action)
-    raise NoMethodError, %[Policy check "#{@action}" not found in #{self.class}] unless respond_to?(@action)
+    if %i(can).index(@action)
+      raise RuntimeError.new('Method name not allowed')
+    end
+
+    unless respond_to?(@action)
+      raise NoMethodError.new(%[Policy check "#{@action}" not found in #{self.class}])
+    end
 
     call *args, &block
   end
@@ -40,48 +33,25 @@ class Policy
 
   private
 
-  # call has to be isolated because specific of error handling
+  # call has to be isolated because of specifics in handling
   def call *args, &block
-    raise Error, 'User is not defined, no access' unless @user
-
     return true if before(@action) == true
-    return true if send(@action, *args) && after(@action) == true
+    return true if send(@action, *args)
 
-    raise Error, 'Access disabled in policy'
+    error 'Access disabled in policy'
   rescue Policy::Error => error
     message = error.message
     message += " - #{self.class}##{@action}"
 
     if block
-      block.call(message)
+      block.call message
       false
     else
-      raise Policy::Error, message
+      error message
     end
   end
 
   def before action
     false
   end
-
-  def after action
-    true
-  end
-
-  def error message
-    raise Policy::Error.new(message)
-  end
-
-  # get current user from globals if globals defined
-  def current_user
-    if defined?(User) && User.respond_to?(:current)
-      User.current
-    elsif defined?(Current) && Current.respond_to?(:user)
-      Current.user
-    elsif user = Thread.current[:current_user]
-      user
-    else
-      raise RuntimeError.new('Current user not found in Policy#current_user')
-    end
-  end
 end

data/lib/egoist/error.rb

@@ -2,3 +2,18 @@ class Policy
   class Error < StandardError
   end
 end
+
+class Policy
+  class << self
+    def error msg
+      raise ::Policy::Error.new(msg)
+    end
+  end
+
+  ###
+
+  def error message
+    raise Policy::Error.new(message)
+  end
+end
+

data/lib/egoist/proxy.rb

@@ -1,4 +1,37 @@
 class Policy
+  class << self
+    # try to load current user
+    def current_user
+      if Thread.current.key?(:current_user)
+        Thread.current[:current_user]
+      elsif defined?(User) && User.respond_to?(:current)
+        User.current
+      elsif defined?(Current) && Current.respond_to?(:user)
+        Current.user
+      else
+        raise RuntimeError.new('Current user not found in Policy#current_user')
+      end
+    end
+
+    def can model = nil, user = nil
+      if model.is_a?(Hash)
+        model, user = model[:model], model[:user]
+      end
+
+      klass = self
+
+      # if we are calling can on Policy class, figure out policy class
+      if self == Policy
+        klass = ('%s_policy' % model.class).classify
+        klass = Object.const_defined?('::%s' % klass) ? klass.constantize : raise('Policy class %s not defined' % klass)
+      end
+
+      klass.new(user: user, model: model).can
+    end
+  end
+
+  ###
+
   class Proxy
     def initialize policy
       @policy = policy
@@ -9,20 +42,23 @@ class Policy
       action = $1
 
       @policy.can?(name, *args)
-      @policy.model || true
-    rescue Policy::Error => error
-      if block_given?
-        yield
-        return nil
-      end
 
       if action == '!'
+        @policy.model || true
+      else
+        true
+      end
+    rescue Policy::Error => error
+      if block_given?
+        yield error
+        nil
+      elsif action == '!'
         raise error
       elsif action == '?'
-        nil
+        false
       else
         raise ArgumentError.new('Bad policy method %s' % name)
       end
     end
   end
-end
+end

data/lib/egoist.rb

@@ -1,7 +1,6 @@
 require_relative 'egoist/base'
 require_relative 'egoist/error'
 require_relative 'egoist/proxy'
-require_relative 'egoist/global'
 
 require_relative 'adapters/controller'
 require_relative 'adapters/model'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: egoist
 version: !ruby/object:Gem::Version
-  version: 0.5.1
+  version: 0.9.0
 platform: ruby
 authors:
 - Dino Reic
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-09-17 00:00:00.000000000 Z
+date: 2022-08-02 00:00:00.000000000 Z
 dependencies: []
 description: Clean, simple explicit and strait-forward policy definitions.
 email: reic.dino@gmail.com
@@ -22,7 +22,6 @@ files:
 - "./lib/egoist.rb"
 - "./lib/egoist/base.rb"
 - "./lib/egoist/error.rb"
-- "./lib/egoist/global.rb"
 - "./lib/egoist/proxy.rb"
 homepage: https://github.com/dux/egoist
 licenses:
@@ -43,7 +42,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.6
+rubygems_version: 3.2.3
 signing_key:
 specification_version: 4
 summary: Ruby access policy library

data/lib/egoist/global.rb

@@ -1,16 +0,0 @@
-# Policy(:application)              -> ApplicationPolicy.can(model: nil, user: current_user)
-# Policy(@post)                     -> PostPolict.can(model: @post, user: current_user)
-# Policy(@post, @user)              -> PostPolict.can(model: @post, user: @user)
-# Policy(model: @post, user: @user) -> PostPolict.can(model: @post, user: @user)
-def Policy model, user=nil
-  if model.is_a?(Hash)
-    user, model = model[:user], model[:model]
-  end
-
-  raise ArgumentError, 'Model not defined' unless model
-
-  klass = model.is_a?(Symbol) ? model : model.class
-  klass = ('%s_policy' % klass).classify.constantize
-
-  klass.new(user: user, model: model).can
-end