rabarber 4.1.2 → 4.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 73050af26c88e78f295c489ee5fc49c782b3950157db2ecd1dd48cdda5b10045
-  data.tar.gz: '08f7da98672774c2198c155be4fb6f795c23e7da72e565631ef0f08207cd2cb9'
+  metadata.gz: 5152a05ff663a5216b2fc2f546bf3b111f3e78a4c44c0088e72f2f73ac89464f
+  data.tar.gz: f6f24ff379055cfd5f4f7e93377a548289fd8f7b30b2f06053fd958f6ab8e5a3
 SHA512:
-  metadata.gz: d1482714aa484c800970ea77a081388a0df4b8945e8fc0931b76e8d6415ed9f88c01ad8834e873d9161a20c7a5f2208b10782fe95857024b1a6069919f83dd47
-  data.tar.gz: '06383d81e91f872419fb567b2ef18991abf6a32263297645725e52b70a9c383a769c72d3657fb9f61a6cc11fb30b2129d813ca63e1a352761f99795f287c86b7'
+  metadata.gz: e31e2c7fa58d42096203d1f8027acea031a720fafe21fd44b179a8c2e66818686ebd55abf2ec0d3229c93fc7e04b514a68fa20feebef25b7e11048caac4e58e7
+  data.tar.gz: 6b7143de48cc95ead0caf5256c9e3d28a54a515f4ea50cf33fc87271d5d13071782fad1926dda698e7842047f7bbed4a2cfe8254be3390af0ab1fde2608a9b30

data/CHANGELOG.md

@@ -1,3 +1,20 @@
+## v4.1.4
+
+### Misc:
+
+- Deprecated `must_have_roles` configuration option
+- Optimized cache invalidation
+
+## v4.1.3
+
+### Bugs:
+
+- Fixed an issue where role presence checking did not work correctly if `must_have_roles` configuration option was enabled
+
+### Misc:
+
+- Added caching for `Rabarber::HasRoles#all_roles` method to improve performance
+
 ## v4.1.2
 
 ### Misc:

data/lib/rabarber/configuration.rb

@@ -20,6 +20,7 @@ module Rabarber
         instance_variable_set(:"@#{method_name}", Rabarber::Input::Types::Boolean.new(
           value, Rabarber::ConfigurationError, "Configuration '#{method_name}' must be a Boolean"
         ).process)
+        ActiveSupport::Deprecation.new("5.0.0", "rabarber").warn("Rabarber’s ‘must_have_roles’ configuration option is deprecated and will be removed in the next major version!") if method_name == :must_have_roles
       end
     end
 

data/lib/rabarber/core/cache.rb

@@ -5,22 +5,18 @@ require "digest/sha2"
 module Rabarber
   module Core
     module Cache
-      CACHE_PREFIX = "rabarber"
-      private_constant :CACHE_PREFIX
-
       module_function
 
-      def fetch(roleable_id, context:, &block)
+      def fetch(uid, &)
         return yield unless enabled?
 
-        Rails.cache.fetch(key_for(roleable_id, context), expires_in: 1.hour, race_condition_ttl: 5.seconds, &block)
+        Rails.cache.fetch(prepare_key(uid), expires_in: 1.hour, race_condition_ttl: 5.seconds, &)
       end
 
-      def delete(*roleable_ids, context:)
+      def delete(*uids)
         return unless enabled?
 
-        keys = roleable_ids.map { |roleable_id| key_for(roleable_id, context) }
-        Rails.cache.delete_multi(keys) if keys.any?
+        Rails.cache.delete_multi(uids.map { prepare_key(_1) }) if uids.any?
       end
 
       def enabled?
@@ -31,9 +27,12 @@ module Rabarber
         Rails.cache.delete_matched(/^#{CACHE_PREFIX}/o)
       end
 
-      def key_for(id, context)
-        "#{CACHE_PREFIX}:#{Digest::SHA2.hexdigest("#{id}#{context.fetch(:context_type)}#{context.fetch(:context_id)}")}"
+      def prepare_key(uid)
+        "#{CACHE_PREFIX}:#{Digest::SHA2.hexdigest(Marshal.dump(uid))}"
       end
+
+      CACHE_PREFIX = "rabarber"
+      private_constant :CACHE_PREFIX
     end
   end
 

data/lib/rabarber/core/null_roleable.rb

@@ -7,6 +7,10 @@ module Rabarber
         []
       end
 
+      def all_roles
+        {}
+      end
+
       def has_role?(*role_names, context: nil) # rubocop:disable Lint/UnusedMethodArgument
         false
       end

data/lib/rabarber/core/rule.rb

@@ -17,10 +17,9 @@ module Rabarber
       end
 
       def roles_permitted?(roleable, controller_instance)
-        resolved_context = resolve_context(controller_instance)
-        return false if Rabarber::Configuration.instance.must_have_roles && roleable.roles(context: resolved_context).empty?
+        return false if Rabarber::Configuration.instance.must_have_roles && roleable.all_roles.empty?
 
-        roles.empty? || roleable.has_role?(*roles, context: resolved_context)
+        roles.empty? || roleable.has_role?(*roles, context: resolve_context(controller_instance))
       end
 
       def dynamic_rules_followed?(controller_instance)

data/lib/rabarber/models/concerns/has_roles.rb

@@ -16,11 +16,11 @@ module Rabarber
 
     def roles(context: nil)
       processed_context = process_context(context)
-      Rabarber::Core::Cache.fetch(roleable_id, context: processed_context) { rabarber_roles.names(context: processed_context) }
+      Rabarber::Core::Cache.fetch([roleable_id, processed_context]) { rabarber_roles.names(context: processed_context) }
     end
 
     def all_roles
-      rabarber_roles.all_names
+      Rabarber::Core::Cache.fetch([roleable_id, :all]) { rabarber_roles.all_names }
     end
 
     def has_role?(*role_names, context: nil)
@@ -102,7 +102,7 @@ module Rabarber
     end
 
     def delete_roleable_cache(context:)
-      Rabarber::Core::Cache.delete(roleable_id, context:)
+      Rabarber::Core::Cache.delete([roleable_id, context], [roleable_id, :all])
     end
 
     def roleable_id

data/lib/rabarber/models/role.rb

@@ -67,7 +67,9 @@ module Rabarber
       private
 
       def delete_roleables_cache(role, context:)
-        Rabarber::Core::Cache.delete(*assigned_to_roleables(role), context:)
+        assigned_to_roleables(role).each_slice(1000) do |ids|
+          Rabarber::Core::Cache.delete(*ids.flat_map { [[_1, context], [_1, :all]] })
+        end
       end
 
       def assigned_to_roleables(role)
@@ -97,8 +99,6 @@ module Rabarber
       record
     end
 
-    private
-
     def delete_assignments
       ActiveRecord::Base.connection.execute(
         ActiveRecord::Base.sanitize_sql(

data/lib/rabarber/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Rabarber
-  VERSION = "4.1.2"
+  VERSION = "4.1.4"
 end

metadata

@@ -1,14 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: rabarber
 version: !ruby/object:Gem::Version
-  version: 4.1.2
+  version: 4.1.4
 platform: ruby
 authors:
 - enjaku4
 - trafium
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-02-21 00:00:00.000000000 Z
+date: 2025-03-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -30,6 +31,7 @@ dependencies:
     - - "<"
       - !ruby/object:Gem::Version
         version: '8.1'
+description:
 email:
 - rabarber_gem@icloud.com
 executables: []
@@ -80,6 +82,7 @@ metadata:
   source_code_uri: https://github.com/enjaku4/rabarber
   changelog_uri: https://github.com/enjaku4/rabarber/blob/main/CHANGELOG.md
   rubygems_mfa_required: 'true'
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -97,7 +100,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.2
+rubygems_version: 3.4.19
+signing_key:
 specification_version: 4
 summary: Simple role-based authorization library for Ruby on Rails
 test_files: []