foreman_salt 8.0.3 → 8.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: b41856693558fa791aa8995c9532ff5fc2c2aee7
-  data.tar.gz: 4d74360561499d39ec1a340d542a3d8c54f5030c
+SHA256:
+  metadata.gz: f6b102a6fc74aec003d945ca7d88f86963c673fc624e8867f84fd59ee061d778
+  data.tar.gz: a1a9409a364838b09777da3477e1037def275fe913544de8766ebd9b2a1d99c5
 SHA512:
-  metadata.gz: a5f3c8092420d2671c6c05b27253cf45691a8ea9fbffad31c9d5bfa4d94c5decf31e4389c33c2b6b8fa99ef188b7fb52dbdc2e3fa5c73935a9e5cbf9e25cc21a
-  data.tar.gz: a3d3f67166b6c124bf47a8d86b05ea286f6ad44251665e0b2057a31b54f79e5d0841285c5effa99bfd33d8f8e968f48b9a4d958abbdf85707f3cc2731ce54b82
+  metadata.gz: 8ad8328433e56b07411ee20fa88f7df325ab40bbd7ba4d2fca557bd8148260929e3027e71319d8f79bbc0795da33f122621a9fb9e13395020af100978e5331c7
+  data.tar.gz: 423f85d580b378fa4bb488bee0d917ed6aa7501dc84e3237a5111dc79f321f9ef15958e00243bfe93f3d0ef6f4b90eadc63e20cdcf621f9a27824dc38562cafa

data/app/controllers/foreman_salt/api/v2/salt_keys_controller.rb

@@ -21,7 +21,7 @@ module ForemanSalt
         param :smart_proxy_id, :identifier_dottable, :required => true
         param :state, String, :required => true, :desc => N_('State can be "accepted" or "rejected"')
         def update
-          case params[:key][:state]
+          case params[:salt_key][:state]
           when 'accepted'
             @key.accept
           when 'rejected'

data/app/controllers/foreman_salt/api/v2/salt_minions_controller.rb

@@ -29,7 +29,7 @@ module ForemanSalt
         param_group :minion
         def update
           params[:minion][:salt_module_ids] = params[:minion].delete(:salt_state_ids) if params[:minion]
-          process_response @salt_minion.update_attributes(params[:minion])
+          process_response @salt_minion.update_attributes(params.require(:minion).permit(:salt_proxy_id, :salt_environment_id, :salt_module_ids => []))
         end
 
         def controller_permission

data/app/controllers/foreman_salt/api/v2/salt_states_controller.rb

@@ -66,7 +66,7 @@ module ForemanSalt
             end
             clean_orphans
           end
-          render :text => states.to_json
+          render :plain => states.to_json
         end
 
         def controller_permission

data/app/controllers/foreman_salt/minions_controller.rb

@@ -16,12 +16,12 @@ module ForemanSalt
 
       enc['environment'] = env
       respond_to do |format|
-        format.html { render :text => "<pre>#{ERB::Util.html_escape(enc.to_yaml)}</pre>" }
-        format.yml  { render :text => enc.to_yaml }
+        format.html { render :plain => "<pre>#{ERB::Util.html_escape(enc.to_yaml)}</pre>" }
+        format.yml  { render :plain => enc.to_yaml }
       end
     rescue
       logger.warn "Failed to generate external nodes for #{@minion} with #{$ERROR_INFO}"
-      render(:text => _('Unable to generate output, Check log files\n'), :status => 412) && return
+      render(:plain => _('Unable to generate output, Check log files\n'), :status => 412) && return
     end
 
     def run

data/app/models/foreman_salt/concerns/host_managed_extensions.rb

@@ -4,8 +4,8 @@ module ForemanSalt
       extend ActiveSupport::Concern
 
       included do
-        has_many :salt_modules, :through => :host_salt_modules, :class_name => '::ForemanSalt::SaltModule'
         has_many :host_salt_modules, :foreign_key => :host_id, :class_name => '::ForemanSalt::HostSaltModule'
+        has_many :salt_modules, :through => :host_salt_modules, :class_name => '::ForemanSalt::SaltModule'
 
         belongs_to :salt_proxy, :class_name => 'SmartProxy'
         belongs_to :salt_environment, :class_name => 'ForemanSalt::SaltEnvironment'

data/app/models/foreman_salt/concerns/hostgroup_extensions.rb

@@ -4,8 +4,8 @@ module ForemanSalt
       extend ActiveSupport::Concern
 
       included do
-        has_many :salt_modules, :through => :hostgroup_salt_modules, :class_name => '::ForemanSalt::SaltModule'
         has_many :hostgroup_salt_modules, :class_name => '::ForemanSalt::HostgroupSaltModule'
+        has_many :salt_modules, :through => :hostgroup_salt_modules, :class_name => '::ForemanSalt::SaltModule'
 
         belongs_to :salt_proxy, :class_name => 'SmartProxy'
         belongs_to :salt_environment, :class_name => 'ForemanSalt::SaltEnvironment'

data/app/models/foreman_salt/salt_environment.rb

@@ -7,8 +7,8 @@ module ForemanSalt
     has_many :hosts, :class_name => '::Host::Managed'
     has_many :hostgroups, :class_name => '::Hostgroup'
 
-    has_many :salt_modules, :through => :salt_module_environments, :before_remove => :remove_from_hosts
     has_many :salt_module_environments
+    has_many :salt_modules, :through => :salt_module_environments, :before_remove => :remove_from_hosts
 
     validates :name, :uniqueness => true, :presence => true, :format => { :with => /\A[\w\d\.]+\z/, :message => N_('is alphanumeric and cannot contain spaces') }
 

data/app/models/foreman_salt/salt_module.rb

@@ -4,14 +4,14 @@ module ForemanSalt
     extend FriendlyId
     friendly_id :name
 
-    has_many :hosts, :through => :host_salt_modules, :class_name => '::Host::Managed'
     has_many :host_salt_modules, :foreign_key => :salt_module_id
+    has_many :hosts, :through => :host_salt_modules, :class_name => '::Host::Managed'
 
-    has_many :hostgroups, :through => :hostgroup_salt_modules
     has_many :hostgroup_salt_modules, :foreign_key => :salt_module_id
+    has_many :hostgroups, :through => :hostgroup_salt_modules
 
-    has_many :salt_environments, :through => :salt_module_environments
     has_many :salt_module_environments
+    has_many :salt_environments, :through => :salt_module_environments
 
     validates :name, :uniqueness => true, :presence => true, :format => { :with => /\A(?:[\w\d\-]+\.{0,1})+[^\.]\z/, :message => N_('must be alphanumeric, can contain periods, dashes, underscores and must not contain spaces') }
 

data/app/services/foreman_salt/smart_proxies/salt_keys.rb

@@ -31,14 +31,12 @@ module ForemanSalt
     end
 
     def accept
-      fail ::Foreman::Exception.new(N_('unable to re-accept an accepted key')) unless state == 'unaccepted'
       proxy = SmartProxy.find(smart_proxy_id)
       Rails.cache.delete("saltkeys_#{proxy.id}") if Rails.env.production?
       ProxyAPI::Salt.new(:url => proxy.url).key_accept name
     end
 
     def reject
-      fail ::Foreman::Exception.new(N_('unable to reject an accepted key')) unless state == 'unaccepted'
       proxy = SmartProxy.find(smart_proxy_id)
       Rails.cache.delete("saltkeys_#{proxy.id}") if Rails.env.production?
       ProxyAPI::Salt.new(:url => proxy.url).key_reject name

data/lib/foreman_salt/version.rb

@@ -1,3 +1,3 @@
 module ForemanSalt
-  VERSION = '8.0.3'
+  VERSION = '8.0.4'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_salt
 version: !ruby/object:Gem::Version
-  version: 8.0.3
+  version: 8.0.4
 platform: ruby
 authors:
 - Stephen Benjamin
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-11-25 00:00:00.000000000 Z
+date: 2018-05-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: deface
@@ -183,14 +183,14 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.2.1
+rubygems_version: 2.7.6
 signing_key: 
 specification_version: 4
 summary: Foreman Plug-in for Salt
 test_files:
 - test/factories/foreman_salt_factories.rb
-- test/functional/api/v2/salt_environments_controller_test.rb
 - test/functional/api/v2/salt_autosign_controller_test.rb
+- test/functional/api/v2/salt_environments_controller_test.rb
 - test/functional/api/v2/salt_keys_controller_test.rb
 - test/functional/api/v2/salt_states_controller_test.rb
 - test/functional/minions_controller_test.rb