certmeister 0.3.2 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c0fae269aadcb7586df47b398aa563c9da8eff81
-  data.tar.gz: af9c8f585af0375c66a39b495579030d5789aab1
+  metadata.gz: 41a8e46ac96ea1df82ccd6bc6d1e7ac813c45b39
+  data.tar.gz: 7d3deab06f303c7725b316db513e1951d7ce2429
 SHA512:
-  metadata.gz: 5abc6a80024b01e68c3baed7d2d00cf002d7cf3e73a40faf583fcaef95e1b8c3a25088df4be5d4969a5733be0bdc9f9c21c79606e7ceca1a8e7a99674460084f
-  data.tar.gz: 3b84043e2f9bec56c7229ee2b35296fdb23dea722e50c966b52c0306204b309fb4bb08fe8930c51ec1a2300cc6e6b2527bf58baf3ccf2362ecf91c2b22ad2a2f
+  metadata.gz: 476a2abd15ebd45ac60facf0c6b3837ae08b3bc11d9785e230ecd94b58acad5aa3abf3a5b5db246be204a9400e1bf2aacc19173ff42f4d3cc721f1edb1f34ef4
+  data.tar.gz: e7385adbdb0288a486897979c64b459b1efb672bbc334502300ac35764bff158a7759060f748e68b5750d013f01b40e8281deb277786d13e06b0c6e16d9c7a7d

data/Gemfile.lock

@@ -1,12 +1,12 @@
 PATH
   remote: .
   specs:
-    certmeister (0.3.2)
-    certmeister-rack (0.3.2)
-      certmeister (= 0.3.2)
+    certmeister (0.4.0)
+    certmeister-rack (0.4.0)
+      certmeister (= 0.4.0)
       rack (~> 1.5)
-    certmeister-redis (0.3.2)
-      certmeister (= 0.3.2)
+    certmeister-redis (0.4.0)
+      certmeister (= 0.4.0)
       redis-sentinel (~> 1.4)
 
 GEM

data/README.md

@@ -56,9 +56,8 @@ If you work at Hetzner and need to release new versions of the gems, do this
 changes):
 
 ```
-bundle exec rake bump:patch # or bump:minor or bump:major
+# edit lib/certmeister/version.rb
 bundle
-git add .semver Gemfile.lock
-git commit -m "Bump to version $(bundle exec semver)"
+git commit -m "Bump to version x.y.z" Gemfile.lock lib/certmeister/version.rb
 bundle exec release
 ```

data/lib/certmeister/policy/chain_all.rb

@@ -6,29 +6,20 @@ module Certmeister
 
     class ChainAll
 
-      def initialize(policys)
-        validate_policys(policys)
-        @policys = policys
+      def initialize(policies)
+        Certmeister::Policy.validate_policies(policies)
+        @policies = policies
       end
 
       def authenticate(request)
         success = Certmeister::Policy::Response.new(true, nil)
-        @policys.inject(success) do |continue, policy|
+        @policies.inject(success) do |continue, policy|
           response = policy.authenticate(request)
           break response unless response.authenticated?
           continue
         end
       end
 
-      private
-
-      def validate_policys(policys)
-        unless policys.is_a?(Enumerable) and policys.respond_to?(:size) and policys.size > 0 and
-               policys.all? { |policy| Certmeister::Policy.validate_authenticate_signature(policy) }
-          raise ArgumentError.new("enumerable collection of policys required")
-        end
-      end
-
     end
 
   end

data/lib/certmeister/policy/chain_any.rb

@@ -0,0 +1,27 @@
+require 'certmeister/policy'
+
+module Certmeister
+
+  module Policy
+
+    class ChainAny
+
+      def initialize(policies)
+        Certmeister::Policy.validate_policies(policies)
+        @policies = policies
+      end
+
+      def authenticate(request)
+        failure = Certmeister::Policy::Response.new(false, "no conditions satisifed")
+        @policies.inject(failure) do |continue, policy|
+          response = policy.authenticate(request)
+          break response if response.authenticated?
+          continue
+        end
+      end
+
+    end
+
+  end
+
+end

data/lib/certmeister/policy.rb

@@ -11,6 +11,13 @@ module Certmeister
       response.respond_to?(:authenticated?) and response.respond_to?(:error)
     end
 
+    def self.validate_policies(policies)
+      unless policies.is_a?(Enumerable) and policies.respond_to?(:size) and policies.size > 0 and
+          policies.all? { |policy| self.validate_authenticate_signature(policy) }
+        raise ArgumentError.new("enumerable collection of policies required")
+      end
+    end
+
   end
 
 end

data/lib/certmeister/version.rb

@@ -1,5 +1,5 @@
 module Certmeister
 
-  VERSION = '0.3.2' unless defined?(VERSION)
+  VERSION = '0.4.0' unless defined?(VERSION)
 
 end

data/spec/certmeister/policy/chain_all_spec.rb

@@ -6,8 +6,8 @@ require 'certmeister/policy/chain_all'
 
 describe Certmeister::Policy::ChainAll do
 
-  it "must be configured with a list of policys" do
-    expected_error = "enumerable collection of policys required"
+  it "must be configured with a list of policies" do
+    expected_error = "enumerable collection of policies required"
     expect { Certmeister::Policy::ChainAll.new }.to raise_error(ArgumentError)
     expect { Certmeister::Policy::ChainAll.new(Certmeister::Policy::Noop.new) }.to raise_error(ArgumentError, expected_error)
     expect { Certmeister::Policy::ChainAll.new([]) }.to raise_error(ArgumentError, expected_error)
@@ -18,18 +18,18 @@ describe Certmeister::Policy::ChainAll do
     expect { policy.authenticate }.to raise_error(ArgumentError)
   end
 
-  it "authenticates a request that all its chained policys authenticate" do
+  it "authenticates a request that all its chained policies authenticate" do
     policy = Certmeister::Policy::ChainAll.new([Certmeister::Policy::Noop.new, Certmeister::Policy::Noop.new])
     response = policy.authenticate({anything: 'something'})
     expect(response).to be_authenticated
   end
 
-  it "refuses a request that any one of its chained policys refuses" do
+  it "refuses a request that any one of its chained policies refuses" do
     refuse_last = Certmeister::Policy::ChainAll.new([ Certmeister::Policy::Noop.new, Certmeister::Policy::Blackhole.new])
     refuse_first = Certmeister::Policy::ChainAll.new([ Certmeister::Policy::Blackhole.new, Certmeister::Policy::Noop.new])
-    policys = [refuse_last, refuse_first]
+    policies = [refuse_last, refuse_first]
 
-    policys.each do |policy|
+    policies.each do |policy|
       response = policy.authenticate({anything: 'something'})
       expect(response).to_not be_authenticated
       expect(response.error).to eql "blackholed"

data/spec/certmeister/policy/chain_any_spec.rb

@@ -0,0 +1,37 @@
+require 'spec_helper'
+require 'certmeister/policy/blackhole'
+require 'certmeister/policy/noop'
+
+require 'certmeister/policy/chain_any'
+
+describe Certmeister::Policy::ChainAny do
+
+  it "must be configured with a list of policies" do
+    expected_error = "enumerable collection of policies required"
+    expect { Certmeister::Policy::ChainAny.new }.to raise_error(ArgumentError)
+    expect { Certmeister::Policy::ChainAny.new(Certmeister::Policy::Noop.new) }.to raise_error(ArgumentError, expected_error)
+    expect { Certmeister::Policy::ChainAny.new([]) }.to raise_error(ArgumentError, expected_error)
+  end
+
+  it "demands a request" do
+    policy = Certmeister::Policy::ChainAny.new([Certmeister::Policy::Noop.new])
+    expect { policy.authenticate }.to raise_error(ArgumentError)
+  end
+
+  it "authenticates a request that any of its chained policies authenticate" do
+    policy = Certmeister::Policy::ChainAny.new([Certmeister::Policy::Blackhole.new, Certmeister::Policy::Noop.new, Certmeister::Policy::Blackhole.new])
+    response = policy.authenticate({anything: 'something'})
+    expect(response).to be_authenticated
+  end
+
+  it "refuses a request that none of its chained policies refuses" do
+    policy = Certmeister::Policy::ChainAll.new([ Certmeister::Policy::Blackhole.new, Certmeister::Policy::Blackhole.new])
+    response = policy.authenticate({anything: 'something'})
+    expect(response).to_not be_authenticated
+    expect(response.error).to eql "blackholed"
+  end
+
+  it "uses the error message of the last encountered refusal in the chain"
+
+end
+

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: certmeister
 version: !ruby/object:Gem::Version
-  version: 0.3.2
+  version: 0.4.0
 platform: ruby
 authors:
 - Sheldon Hearn
@@ -93,6 +93,7 @@ files:
 - lib/certmeister/policy.rb
 - lib/certmeister/policy/blackhole.rb
 - lib/certmeister/policy/chain_all.rb
+- lib/certmeister/policy/chain_any.rb
 - lib/certmeister/policy/domain.rb
 - lib/certmeister/policy/existing.rb
 - lib/certmeister/policy/fcrdns.rb
@@ -109,6 +110,7 @@ files:
 - spec/certmeister/in_memory_store_spec.rb
 - spec/certmeister/policy/blackhole_spec.rb
 - spec/certmeister/policy/chain_all_spec.rb
+- spec/certmeister/policy/chain_any_spec.rb
 - spec/certmeister/policy/domain_spec.rb
 - spec/certmeister/policy/existing_spec.rb
 - spec/certmeister/policy/fcrdns_spec.rb
@@ -153,6 +155,7 @@ test_files:
 - spec/certmeister/in_memory_store_spec.rb
 - spec/certmeister/policy/blackhole_spec.rb
 - spec/certmeister/policy/chain_all_spec.rb
+- spec/certmeister/policy/chain_any_spec.rb
 - spec/certmeister/policy/domain_spec.rb
 - spec/certmeister/policy/existing_spec.rb
 - spec/certmeister/policy/fcrdns_spec.rb