foreman_openscap 0.5.0 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f4717f3191e1244335417139e2dddef5297b612d
-  data.tar.gz: 54ac0f517e808a186d74a6a42a03f07a79565abf
+  metadata.gz: 121b364917afa0cdfdc713d1faf79d3ab6ccd3b9
+  data.tar.gz: 28b664428cc89f6b6f5f668d94b4d9df84b73ea0
 SHA512:
-  metadata.gz: 2dbb8f627fdb059e1099c4f95288bfdc7b4ee9e2eef011cec3c45c9c7419ee8ccf7e99813894d46ba659c8d2cfc88d44cc86ebec0a5e24f91312489478133193
-  data.tar.gz: c74cd26daf940251810f9aae28728897da0d8d35848939476c2e90a882ae569e5a698835c11b24d669539c187c2bd6526a3c46f6e319bf5b251382190c09ae11
+  metadata.gz: d5fe4ad1b975934842292b33f3ccfb147d69054cdfb0c4d65c73c31cd905ebb863ba11cdab59d3b7f5a6dbd2fcbfd3e575ec9d5257d9ce8eccf5c0ffe205d2fb
+  data.tar.gz: fad85cd933a8799fd6bb064f816c4770531ccd97e72c2d0e052ddeb63c8f45972adf087f0dc193e90eae6f894fad582043cf31f203b3f4f2d1ef40e4a83ab2f7

data/app/assets/javascripts/foreman_openscap/arf_reports.js

@@ -0,0 +1,16 @@
+function buildArfModal(element, url) {
+  var url = url + "?" + $.param({arf_report_ids: $.foremanSelectedHosts});
+  var title = $(element).attr('data-dialog-title');
+  $('#confirmation-modal .modal-header h4').text(title);
+  $("#confirmation-modal .modal-body").load(url + " #content",
+      function(response, status, xhr) {
+        $('#submit_multiple').val('');
+        var b = $("#confirmation-modal .btn-primary");
+        if ($(response).find('#content form select').length > 0)
+          b.addClass("disabled").attr("disabled", true);
+        else
+          b.removeClass("disabled").attr("disabled", false);
+        $('#confirmation-modal').modal();
+        });
+  return false;
+}

data/app/assets/javascripts/foreman_openscap/openscap_proxy.js

@@ -0,0 +1,52 @@
+function updateOpenscapProxy(element){
+  var id = $("form").data('id')
+  var url = $(element).attr('data-url');
+  var data = $("form").serialize().replace('method=put', 'method=post');
+  if (url.match('hostgroups')) {
+    data = data + '&hostgroup_id=' + id
+  } else {
+    data = data + '&host_id=' + id
+  }
+
+  toggleErrorText("");
+  $(element).indicator_show();
+  $.ajax({
+    type: 'post',
+    url:  url,
+    data: data,
+    error: function(response) {
+      var text = $(response.responseText).find('.form-group.has-error').find('.help-block.help-inline').text();
+      addOpenscapProxyError(text);
+      $(element).indicator_hide();
+    },
+    success: function(response) {
+      removeOpenscapProxyError();
+      $('#puppetclasses_parameters').replaceWith($(response).find("#puppetclasses_parameters"));
+      $(element).indicator_hide();
+    }
+  })
+}
+
+function findOpenscapProxyFormGroup(){
+  return $('form').find("label[for='openscap_proxy_id']").parents(".form-group").first();
+}
+
+function addOpenscapProxyError(text){
+  var formGroup = findOpenscapProxyFormGroup();
+  $(formGroup).addClass("has-error");
+  toggleErrorText(text);
+}
+
+function removeOpenscapProxyError(){
+  var formGroup = findOpenscapProxyFormGroup();
+  $(formGroup).removeClass("has-error");
+  toggleErrorText("");
+}
+
+function toggleErrorText(text){
+  if(text){
+    $(findOpenscapProxyFormGroup()).find(".help-block.help-inline").append('<span id="openscap_error">' + text + '</span>');
+  } else {
+    $(findOpenscapProxyFormGroup()).find("#openscap_error").remove();
+  }
+}

data/app/assets/javascripts/foreman_openscap/reports.js

@@ -0,0 +1,6 @@
+function showReportDetails(log_id, event) {
+  var showDetails = $('#details-' + log_id)
+  showDetails.toggle();
+  showDetails.is(':visible') ? $(event).find('span').attr('class', 'glyphicon glyphicon-collapse-up') : $(event).find('span').attr('class', 'glyphicon glyphicon-collapse-down');
+}
+

data/app/assets/stylesheets/foreman_openscap/policy.css

@@ -4,4 +4,5 @@
 
 .hide-pane {
   display: none;
-}
+}
+

data/app/assets/stylesheets/foreman_openscap/reports.css

@@ -0,0 +1,3 @@
+#report_log a.more-info {
+  color: #CAC5C5;
+}

data/app/controllers/api/v2/compliance/arf_reports_controller.rb

@@ -20,14 +20,14 @@ module Api
 
         add_smart_proxy_filters :create, :features => 'Openscap'
 
-        before_filter :find_resource, :only => %w{show destroy}
+        before_filter :find_resource, :only => %w(show destroy)
 
         def resource_name
           '::ForemanOpenscap::ArfReport'
         end
 
         def get_resource
-          instance_variable_get :"@arf_report" or raise 'no resource loaded'
+          instance_variable_get :"@arf_report" or fail 'no resource loaded'
         end
 
         resource_description do

data/app/controllers/api/v2/compliance/policies_controller.rb

@@ -5,7 +5,7 @@ module Api::V2
 
       add_smart_proxy_filters :content, :features => 'Openscap'
 
-      before_filter :find_resource, :except => %w{index create}
+      before_filter :find_resource, :except => %w(index create)
 
       skip_after_filter :log_response_body, :only => [:content]
 
@@ -14,7 +14,7 @@ module Api::V2
       end
 
       def get_resource
-        instance_variable_get :"@policy" or raise 'no resource loaded'
+        instance_variable_get :"@policy" or fail 'no resource loaded'
       end
 
       def policy_url(policy = nil)
@@ -94,10 +94,10 @@ module Api::V2
 
       def action_permission
         case params[:action]
-          when 'content'
-            :view
-          else
-            super
+        when 'content'
+          :view
+        else
+          super
         end
       end
     end

data/app/controllers/api/v2/compliance/scap_contents_controller.rb

@@ -1,14 +1,14 @@
 module Api::V2
   module Compliance
     class ScapContentsController < ::Api::V2::BaseController
-      before_filter :find_resource, :except => %w{index create}
+      before_filter :find_resource, :except => %w(index create)
 
       def resource_name
         '::ForemanOpenscap::ScapContent'
       end
 
       def get_resource
-        instance_variable_get :"@scap_content" or raise 'no resource loaded'
+        instance_variable_get :"@scap_content" or fail 'no resource loaded'
       end
 
       resource_description do

data/app/controllers/arf_reports_controller.rb

@@ -1,14 +1,15 @@
 class ArfReportsController < ApplicationController
   include Foreman::Controller::AutoCompleteSearch
 
-  before_filter :find_by_id, :only => [:show, :show_html, :destroy, :parse_html, :parse_bzip]
+  before_filter :find_arf_report, :only => [:show, :show_html, :destroy, :parse_html, :parse_bzip]
+  before_filter :find_multiple, :only => [:delete_multiple, :submit_delete_multiple]
 
   def model_of_controller
     ::ForemanOpenscap::ArfReport
   end
 
   def index
-    @arf_reports = resource_base.includes(:asset)
+    @arf_reports = resource_base.includes(:host => [:policies, :last_report_object, :host_statuses])
       .search_for(params[:search], :order => params[:order])
       .paginate(:page => params[:page], :per_page => params[:per_page])
   end
@@ -41,20 +42,55 @@ class ArfReportsController < ApplicationController
     if @arf_report.destroy
       process_success(:success_msg => (_("Successfully deleted Arf report.")), :success_redirect => arf_reports_path)
     else
-      process_error
+      process_error(:error_msg => _("Failed to delete Arf Report for host #{@arf_report.host.name} reported at #{@arf_report.reported_at}"))
+    end
+  end
+
+  def delete_multiple
+  end
+
+  def submit_delete_multiple
+    failed_deletes = @arf_reports.reject(&:destroy).count
+    if failed_deletes > 0
+      process_error(:error_msg => (_("Failed to delete %s compliance reports") % failed_deletes),
+                    :error_redirect => arf_reports_path)
+    else
+      process_success(:success_msg => (_("Successfully deleted %s compliance reports") % @arf_reports.count),
+                      :success_redirect => arf_reports_path)
     end
   end
 
   private
 
-  def find_by_id
-    @arf_report = resource_base.find(params[:id])
+  def find_arf_report
+    @arf_report = resource_base.includes(:logs => [:message, :source]).find(params[:id])
+  end
+
+  def find_multiple
+    if params[:arf_report_ids].present?
+      @arf_reports = ::ForemanOpenscap::ArfReport.where(:id => params[:arf_report_ids])
+      if @arf_reports.empty?
+        error _('No complince reports were found.')
+        redirect_to(arf_reports_path) and return false
+      end
+    else
+      error _('No complince reports selected')
+      redirect_to(arf_reports_path) and return false
+    end
+    return @arf_reports
+  rescue => e
+    error _("Something went wrong while selecting compliance reports - %s") % (e)
+    logger.debug e.message
+    logger.debug e.backtrace.join("\n")
+    redirect_to arf_reports_path and return false
   end
 
   def action_permission
     case params[:action]
     when 'show_html', 'parse_html', 'parse_bzip'
       :view
+    when 'delete_multiple', 'submit_delete_multiple'
+      :destroy
     else
       super
     end

data/app/controllers/concerns/foreman_openscap/hostgroups_controller_extensions.rb

@@ -0,0 +1,20 @@
+module ForemanOpenscap
+  module HostgroupsControllerExtensions
+    extend ActiveSupport::Concern
+
+    included do
+      alias_method_chain :load_vars_for_ajax, :openscap
+      alias_method_chain :inherit_parent_attributes, :openscap
+    end
+
+    def load_vars_for_ajax_with_openscap
+      load_vars_for_ajax_without_openscap
+      @openscap_proxy = @hostgroup.openscap_proxy
+    end
+
+    def inherit_parent_attributes_with_openscap
+      inherit_parent_attributes_without_openscap
+      @hostgroup.openscap_proxy ||= @parent.openscap_proxy
+    end
+  end
+end

data/app/controllers/concerns/foreman_openscap/hosts_common_controller_extensions.rb

@@ -0,0 +1,45 @@
+module ForemanOpenscap
+  module HostsCommonControllerExtensions
+    extend ActiveSupport::Concern
+
+    def openscap_proxy_changed
+      model = if params[param_id_key].blank?
+                controller_name.classify.constantize.new(params[param_model_key])
+              else
+                resource_base.find(params[param_id_key])
+              end
+      proxy_id = params[param_model_key][:openscap_proxy_id]
+      instance_variable_set("@#{param_model_key}", model)
+      if proxy_id.blank?
+        render :partial => "form"
+      else
+        begin
+          instance_variable_get("@#{param_model_key}").update_scap_client_params(proxy_id)
+          render :partial => "form"
+        rescue => e
+          instance_variable_get("@#{param_model_key}").errors.add(:openscap_proxy_id, e.message)
+          render :partial => "form", :status => 422
+        end
+      end
+    end
+
+    def action_permission
+      case params[:action]
+      when 'openscap_proxy_changed'
+        :edit
+      else
+        super
+      end
+    end
+
+    private
+
+    def param_id_key
+      "#{param_model_key}_id"
+    end
+
+    def param_model_key
+      controller_name.singularize
+    end
+  end
+end

data/app/controllers/concerns/foreman_openscap/hosts_controller_extensions.rb

@@ -0,0 +1,17 @@
+module ForemanOpenscap
+  module HostsControllerExtensions
+    extend ActiveSupport::Concern
+
+    included do
+      alias_method_chain :process_hostgroup, :openscap
+      self::AJAX_REQUESTS << 'openscap_proxy_changed'
+    end
+
+    def process_hostgroup_with_openscap
+      @hostgroup = Hostgroup.find(params[:host][:hostgroup_id]) if params[:host][:hostgroup_id].to_i > 0
+      return head(:not_found) unless @hostgroup
+      @openscap_proxy = @hostgroup.openscap_proxy
+      process_hostgroup_without_openscap
+    end
+  end
+end

data/app/controllers/policies_controller.rb

@@ -134,10 +134,10 @@ class PoliciesController < ApplicationController
 
   def action_permission
     case params[:action]
-      when 'parse'
-        :view
-      else
-        super
+    when 'parse'
+      :view
+    else
+      super
     end
   end
 end

data/app/helpers/arf_report_dashboard_helper.rb

@@ -1,8 +1,8 @@
 module ArfReportDashboardHelper
   COLORS = {
-      :passed => '#89A54E',
-      :failed => '#AA4643',
-      :othered => '#DB843D',
+    :passed => '#89A54E',
+    :failed => '#AA4643',
+    :othered => '#DB843D',
   }
 
   def reports_breakdown_chart(report, options = {})
@@ -10,9 +10,9 @@ module ArfReportDashboardHelper
     [[:failed, _('Failed')],
      [:passed, _('Passed')],
      [:othered, _('Othered')],
-    ].each { |i|
+    ].each do |i|
       data << {:label => i[1], :data => report[i[0]], :color => COLORS[i[0]]}
-    }
+    end
     flot_pie_chart 'overview', _('Compliance reports breakdown'), data, options
   end
 end

data/app/helpers/arf_reports_helper.rb

@@ -34,4 +34,32 @@ module ArfReportsHelper
           end
     "class='label label-#{tag}'".html_safe
   end
+
+  def severity_tag(level)
+    tag = case level.downcase.to_sym
+          when :low
+            "info"
+          when :medium
+            "warning"
+          when :high
+            "danger"
+          else
+            "default"
+          end
+    "class='label label-#{tag}'".html_safe
+  end
+
+  def multiple_actions_arf_report
+    actions = [
+      [_('Delete reports'), delete_multiple_arf_reports_path]
+    ]
+  end
+
+  def multiple_actions_arf_report_select
+    select_action_button(_("Select Action"), {:id => 'submit_multiple'},
+      multiple_actions_arf_report.map do |action|
+        link_to_function(action[0], "buildArfModal(this, '#{action[1]}')",
+         :'data-dialog-title' => _("%s - The following compliance reports are about to be changed") % action[0])
+      end.flatten)
+  end
 end

data/app/helpers/compliance_hosts_helper.rb

@@ -5,14 +5,16 @@ module ComplianceHostsHelper
     [[:passed, _('Passed')],
      [:failed, _('Failed')],
      [:othered, _('Other')],
-    ].each { |i|
+    ].each do |i|
       data << {:label => i[1], :data => report[i[0]], :color => ArfReportDashboardHelper::COLORS[i[0]]}
-    }
+    end
     flot_pie_chart 'overview', _('Compliance reports breakdown'), data, options
   end
 
   def host_arf_reports_chart(policy_id)
-    passed, failed, othered, = [], [], []
+    passed = []
+    failed = []
+    othered = []
     @host.arf_reports.of_policy(policy_id).each do |report|
       passed  << [report.created_at.to_i*1000, report.passed]
       failed  << [report.created_at.to_i*1000, report.failed]

data/app/helpers/policies_helper.rb

@@ -9,7 +9,7 @@ module PoliciesHelper
     scap_contents = ::ForemanOpenscap::ScapContent.all
     if scap_contents.length > 1
       select_f form, :scap_content_id, scap_contents, :id, :title,
-               {:include_blank => _("Choose existing SCAP Content")} ,
+               {:include_blank => _("Choose existing SCAP Content")},
                {:label => _("SCAP Content"),
                 :onchange => 'scap_content_selected(this);',
                 :'data-url' => method_path('scap_content_selected')}
@@ -44,7 +44,7 @@ module PoliciesHelper
       content_tag(:div, :class => "form-actions") do
         text    = overwrite ? overwrite : _("Submit")
         options = {:class => "btn btn-primary"}
-        options.merge! :'data-id' => form_to_submit_id(form) unless options.has_key?(:'data-id')
+        options.merge! :'data-id' => form_to_submit_id(form) unless options.key?(:'data-id')
         previous = form.object.first_step? ? ' ' : previous_link(form)
         cancel_and_submit = content_tag(:div, :class => "pull-right") do
           link_to(_("Cancel"), args[:cancel_path], :class => "btn btn-default") + ' ' +
@@ -67,6 +67,6 @@ module PoliciesHelper
   end
 
   def days_of_week_hash
-    Hash[*Date::DAYNAMES.map{ |day| [day.downcase, day]}.flatten]
+    Hash[*Date::DAYNAMES.map { |day| [day.downcase, day] }.flatten]
   end
 end