chef_fixie 0.2.0 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 1cc981bb23297717dac0de3676f42ea4422016ad
-  data.tar.gz: cfdc771777545508b0d3d195adc668e824ffe4f2
+SHA256:
+  metadata.gz: 87bfff60eb67f1a48a3d9b706e3d4458b325bf8d3d33c7d9952446d3b155d48d
+  data.tar.gz: 7587870c7e37e9f99b199874b061064f7b72a3b742165bfb3057694daf6f3960
 SHA512:
-  metadata.gz: 4241554e99a513b1461bd8de38a199cab655f3ac526c276565c7b7ca2a700fc363d183d8b89bd7f5f12308c0c4fe3a82706c781bccc90ba0772c8a94b73a1dd7
-  data.tar.gz: 1384e2f28b152c93c83a398940d2bbf5f6b1d052bca96817e429aff860bc22d2e7df84edbcad6f07d439f14a11639044b285c0f8b03caaffb597b53dce218a9f
+  metadata.gz: 99b893563cd5061f724dd119fa2f35afc051dbe2fb7b764a127175aeff6afd196f0db11ba70a4b0f41988291a5ac7310f6aa8d4c2092bf7950dba7b6be62f219
+  data.tar.gz: db41d88315375e9102c6d6211eb41029ec197efb50fafeff4b9a0fc918a27f3530ea3cb0ae500039fbc6089ea7ec165f522fb250d8752a237a90a44969c054de

data/bin/chef_fixie

@@ -1,5 +1,5 @@
 #!/usr/bin/env ruby
 
-require_relative '../lib/chef_fixie/console'
+require_relative "../lib/chef_fixie/console"
 
 ChefFixie::Console.start

data/doc/BulkFixup.md

@@ -15,7 +15,7 @@ If a key group is deleted (such as users)
 ```ruby
 users_group.ace_add([:create,:read,:update,:delete], org.groups['admins'])
 users_group.ace_add([:create,:read,:update,:delete], USERS['pivotal'])
-``
+```
 
 * Restore users to the appropriate container ACLs
 ```ruby

data/doc/CommonTasks.md

@@ -9,12 +9,23 @@ points
 
 First of all, run the automated org association checker:
 
-fixie:0 > Fixie::CheckOrgAssociations.check_associations("acme")
-Org acme is ok (6 users)
+    fixie:0 > Fixie::CheckOrgAssociations.check_associations("acme")
+    Org acme is ok (6 users)
 
 If it reports a problem with a user, you may be able to fix it
 automatically:
 
-fixie:0 > Fixie::CheckOrgAssociations.fix_association("acme", "mary")
+    fixie:0 > Fixie::CheckOrgAssociations.fix_association("acme", "mary")
 
 This might need to be run multiple times to fix all of the errors.
+
+
+Removing a user completely from an org
+-----------
+
+    [1] fixie(main)> ChefFixie::CheckOrgAssociations.remove_association('the_org', 'the_user')
+
+This removes the user from the org, and removes them from all org
+groups. However, if the user has been individually added to an ACL we
+don't fix that up; it would require enumeration of the whole org, and
+that hasn't been implemented.

data/lib/chef_fixie.rb

@@ -1,5 +1,5 @@
 #
-# Copyright (c) 2014-2015 Chef Software Inc. 
+# Copyright (c) 2014-2015 Chef Software Inc.
 # License :: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -16,13 +16,13 @@
 #
 # Author: Mark Anderson <mark@chef.io>
 
-require 'sequel'
-require_relative 'chef_fixie/config'
-require_relative 'chef_fixie/sql'
-require_relative 'chef_fixie/sql_objects'
+require "sequel"
+require_relative "chef_fixie/config"
+require_relative "chef_fixie/sql"
+require_relative "chef_fixie/sql_objects"
 
 # This doesn't work because of initialization order, figure it out.
-require_relative 'chef_fixie/check_org_associations'
-require_relative 'chef_fixie/bulk_edit_permissions'
+require_relative "chef_fixie/check_org_associations"
+require_relative "chef_fixie/bulk_edit_permissions"
 
 Sequel.extension :inflector

data/lib/chef_fixie/authz_mapper.rb

@@ -1,5 +1,5 @@
 #
-# Copyright (c) 2014-2015 Chef Software Inc. 
+# Copyright (c) 2014-2015 Chef Software Inc.
 # License :: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -17,9 +17,9 @@
 # Author: Mark Anderson <mark@chef.io>
 #
 
-require 'pp'
-require_relative 'config'
-require_relative 'authz_objects'
+require "pp"
+require_relative "config"
+require_relative "authz_objects"
 
 module ChefFixie
   module AuthzMapper
@@ -32,7 +32,7 @@ module ChefFixie
     #
     # Much of this might be better folded up into a sql stored procedure
     #
-   
+
     def self.included(base)
       base.extend(ClassMethods)
     end
@@ -44,9 +44,9 @@ module ChefFixie
       if objects.count == 1
         object = objects.first
         name = object.name
-        scope = 
-          if object.respond_to?(:org_id) 
-              ChefFixie::Sql::Orgs.org_guid_to_name(object.org_id)
+        scope =
+          if object.respond_to?(:org_id)
+            ChefFixie::Sql::Orgs.org_guid_to_name(object.org_id)
           else
             :global
           end
@@ -57,12 +57,12 @@ module ChefFixie
     end
 
     class ReverseMapper
-      attr_reader :names,:by_type, :instance
-                     
+      attr_reader :names, :by_type, :instance
+
       def initialize
         # name of object map
         @names ||= {}
-        @by_type ||= {:actor=>{}, :container=>{}, :group=>{}, :object=>{}}
+        @by_type ||= { :actor => {}, :container => {}, :group => {}, :object => {} }
         # maps class to a pre-created instance for efficiency
         @instance ||= {}
       end
@@ -75,14 +75,14 @@ module ChefFixie
         names[name] = klass
         by_type[type][name] = klass
       end
-      
+
       def dump
         pp names
       end
 
-      def authz_to_name(authz_id, ctype=nil)
+      def authz_to_name(authz_id, ctype = nil)
         types = if ctype.nil?
-                  AuthzUtils::Types
+                  AuthzUtils::TYPES
                 else
                   [ctype]
                 end
@@ -92,52 +92,50 @@ module ChefFixie
             return result if result != :unknown
           end
         end
-        return :unknown
+        :unknown
       end
     end
 
     def self.mapper
       @mapper ||= ReverseMapper.new
     end
-    
+
     def self.register(klass, name, type)
-      self.mapper.register(klass,name,type)
+      mapper.register(klass, name, type)
     end
 
     # Translates the json from authz for group membership and acls into a human readable form
     # This makes some assumptions about the shape of the data structure, but works well enough to
     # be quite useful
     def self.struct_to_name(s)
-      mapper = AuthzMapper::mapper      
+      mapper = AuthzMapper.mapper
       if s.kind_of?(Hash)
         s.keys.inject({}) do |h, k|
           v = s[k]
           if v.kind_of?(Array)
             case k
-            when 'actors'
-              h[k] = v.map {|a| mapper.authz_to_name(a,:actor) } #.sort We should sort these, but the way we're returning unknown causes sort 
-            when 'groups'
-              h[k] = v.map {|a| mapper.authz_to_name(a,:group) } #.sort to fail
+            when "actors"
+              h[k] = v.map { |a| mapper.authz_to_name(a, :actor) } #.sort We should sort these, but the way we're returning unknown causes sort
+            when "groups"
+              h[k] = v.map { |a| mapper.authz_to_name(a, :group) } #.sort to fail
             else
               h[k] = v
             end
           else
-            h[k] = self.struct_to_name(v)
+            h[k] = struct_to_name(v)
           end
           h
         end
       end
     end
-    
+
     module ClassMethods
       # TODO: We should be able to automatically figure out the type somehow.
       # At minimum should figure out a self check
       def register_authz(name, type)
-        AuthzMapper::register(self,name,type)
+        AuthzMapper.register(self, name, type)
       end
     end
-    
+
   end
 end
-
-  

data/lib/chef_fixie/authz_objects.rb

@@ -17,16 +17,16 @@
 # Author: Mark Anderson <mark@chef.io>
 #
 
-require 'pp'
-require 'ffi_yajl'
-require 'chef/http'
+require "pp"
+require "ffi_yajl"
+require "chef/http"
 
-require_relative 'config'
+require_relative "config"
 
 module ChefFixie
 
   class AuthzApi
-    def initialize(user=nil)
+    def initialize(user = nil)
       @requestor_authz = user ? user : ChefFixie.configure { |x| x.superuser_id }
       @auth_uri ||= ChefFixie.configure { |x| x.authz_uri }
       @rest = Chef::HTTP.new(@auth_uri)
@@ -42,38 +42,41 @@ module ChefFixie
 
     def get(resource)
       result = @rest.get(resource,
-                         'Content-Type'=>'application/json',
-                         'Accept'=>'application/json',
-                         'X-Ops-Requesting-Actor-Id'=>@requestor_authz)
+                         "Content-Type" => "application/json",
+                         "Accept" => "application/json",
+                         "X-Ops-Requesting-Actor-Id" => @requestor_authz)
       FFI_Yajl::Parser.parse(result)
     end
+
     def put(resource, data)
-      result = @rest.put(resource, self.json_helper(data),
-                         'Content-Type'=>'application/json',
-                         'Accept'=>'application/json',
-                         'X-Ops-Requesting-Actor-Id'=>@requestor_authz)
+      result = @rest.put(resource, json_helper(data),
+                         "Content-Type" => "application/json",
+                         "Accept" => "application/json",
+                         "X-Ops-Requesting-Actor-Id" => @requestor_authz)
       FFI_Yajl::Parser.parse(result)
     end
+
     def post(resource, data)
-      result = @rest.post(resource, self.json_helper(data),
-                          'Content-Type'=>'application/json',
-                          'Accept'=>'application/json',
-                          'X-Ops-Requesting-Actor-Id'=>@requestor_authz)
+      result = @rest.post(resource, json_helper(data),
+                          "Content-Type" => "application/json",
+                          "Accept" => "application/json",
+                          "X-Ops-Requesting-Actor-Id" => @requestor_authz)
       FFI_Yajl::Parser.parse(result)
     end
+
     def delete(resource)
       result = @rest.delete(resource,
-                            'Content-Type'=>'application/json',
-                            'Accept'=>'application/json',
-                            'X-Ops-Requesting-Actor-Id'=>@requestor_authz)
+                            "Content-Type" => "application/json",
+                            "Accept" => "application/json",
+                            "X-Ops-Requesting-Actor-Id" => @requestor_authz)
       FFI_Yajl::Parser.parse(result)
     end
 
   end
 
   module AuthzUtils
-    Types = [:object,:actor,:group,:container] # order is an attempt to optimize by most probable.
-    Actions = [:create, :read, :update, :delete, :grant]
+    TYPES = [:object, :actor, :group, :container] # order is an attempt to optimize by most probable.
+    ACTIONS = [:create, :read, :update, :delete, :grant]
 
     def to_resource(t)
       # This is a rails thing... t.to_s.pluralize
@@ -81,20 +84,20 @@ module ChefFixie
     end
 
     def get_type(id)
-      Types.each do |t|
+      TYPES.each do |t|
         begin
-          r = AuthzApi.get("#{self.to_resource(t)}/#{id}")
+          r = AuthzApi.get("#{to_resource(t)}/#{id}")
           return t
-        rescue RestClient::ResourceNotFound=>e
+        rescue RestClient::ResourceNotFound => e
           # expected if not found
         end
       end
-      return :none
+      :none
     end
 
     def check_action(action)
       # TODO Improve; stack trace isn't the best way to communicate with the user
-      raise "#{action} not one of #{Actions.join(', ')} " if !Actions.member?(action)
+      raise "#{action} not one of #{ACTIONS.join(', ')} " if !ACTIONS.member?(action)
     end
 
     def check_actor_or_group(a_or_g)
@@ -102,7 +105,7 @@ module ChefFixie
     end
 
     def resourcify_actor_or_group(a_or_g)
-      return a_or_g if ["actors", "groups"].member?(a_or_g)
+      return a_or_g if %w{actors groups}.member?(a_or_g)
       check_actor_or_group(a_or_g)
       to_resource(a_or_g)
     end
@@ -131,10 +134,9 @@ module ChefFixie
     end
 
     def authz_api
-       @@authz_apiAsSuperUser ||= AuthzApi.new
+      @@authz_api_as_superuser ||= AuthzApi.new
     end
 
-
     # we expect to be mixed in with a class that has the authz_id method
     def prefix
       "#{to_resource(type)}/#{authz_id}"
@@ -145,9 +147,14 @@ module ChefFixie
       [:unparsed, result] # todo figure this out in more detail
     end
 
+    def authz_delete
+      authz_api.delete(prefix)
+    end
+
     def acl_raw
       authz_api.get("#{prefix}/acl")
     end
+
     # Todo: filter this by scope and type
     def acl
       ChefFixie::AuthzMapper.struct_to_name(acl_raw)
@@ -161,11 +168,11 @@ module ChefFixie
       [resource, ace]
     end
 
-
     def ace_raw(action)
-      resource,ace = ace_get_util(action)
+      resource, ace = ace_get_util(action)
       ace
     end
+
     # Todo: filter this by scope and type
     def ace(action)
       ChefFixie::AuthzMapper.struct_to_name(ace_raw(action))
@@ -173,14 +180,11 @@ module ChefFixie
 
     def expand_actions(action)
       if action == :all
-        action = AuthzUtils::Actions
+        action = AuthzUtils::ACTIONS
       end
       action.is_a?(Array) ? action : [action]
-    end
-
-
+    end # add actor or group to acl
 
-    # add actor or group to acl
     def ace_add_raw(action, actor_or_group, entity)
       # groups or actors
       a_or_g_resource = resourcify_actor_or_group(actor_or_group)
@@ -190,9 +194,10 @@ module ChefFixie
       ace[a_or_g_resource].uniq!
       authz_api.put("#{resource}", ace)
     end
+
     def ace_add(action, entity)
       actions = expand_actions(action)
-      actions.each {|a| ace_add_raw(a, entity.type, entity) }
+      actions.each { |a| ace_add_raw(a, entity.type, entity) }
     end
 
     def ace_delete_raw(action, actor_or_group, entity)
@@ -207,7 +212,7 @@ module ChefFixie
 
     def ace_delete(action, entity)
       actions = expand_actions(action)
-      actions.each {|a| ace_delete_raw(a, entity.type, entity) }
+      actions.each { |a| ace_delete_raw(a, entity.type, entity) }
     end
 
     def ace_member?(action, entity)
@@ -216,7 +221,6 @@ module ChefFixie
       ace[a_or_g_resource].member?(entity.authz_id)
     end
 
-
     def acl_add_from_object(object)
       src = object.acl_raw
 
@@ -254,15 +258,21 @@ module ChefFixie
     def group_raw
       authz_api.get("#{prefix}")
     end
+
     # Todo: filter this by scope and type
     def group
       ChefFixie::AuthzMapper.struct_to_name(group_raw)
     end
 
+    def list
+      group
+    end
+
     def group_add_raw(actor_or_group, entity)
       entity_resource = to_resource(actor_or_group)
-      authz_api.put("#{prefix}/#{entity_resource}/#{entity.authz_id}",{})
+      authz_api.put("#{prefix}/#{entity_resource}/#{entity.authz_id}", {})
     end
+
     def group_add(entity)
       group_add_raw(entity.type, entity)
     end
@@ -278,7 +288,7 @@ module ChefFixie
 
     def member?(entity)
       members = group_raw
-      return members[resourcify_actor_or_group(entity.type)].member?(entity.authz_id)
+      members[resourcify_actor_or_group(entity.type)].member?(entity.authz_id)
     end
   end