foreman_webhooks 3.2.1 → 3.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: aaeb5b54bb4d6679430fefcb7d569bf57d19d5df1aacc73b34dae585766878b1
-  data.tar.gz: df3d0865057e2bd34877d7a1cad67303e63c369d691efeda0e149f3ca7a27581
+  metadata.gz: 457b89d953e3410524cd3ad5aee4d63ca47192037523ee2c606e69d291e0e8e2
+  data.tar.gz: 9e43a93a35c44a7e8d0d3fdcecef2f4196120645ffe9b0517fba5a3ded221620
 SHA512:
-  metadata.gz: a6eba6fd3256c144a37615dea8d61519d3878c9ea8a3b0ef8c36f3e45d61b5d1b911ece90e5027d5af6aadda94c79ce272f6051eeb7ce626e647aca520b49326
-  data.tar.gz: b19adb7f561f38537f5826ceca7d0e53ca1617e049071912e0504c62f8a31550f6e117f46cfa47901d2b05fb159ad589baaf0c2b469b91a0962a515451fd8c12
+  metadata.gz: 7dca14e67eb8d98c5677ecdbb0e11a1a580fa9d6499f4c164aa263668a7709cee10441cd552fb4f0d2b9a74965520d0e4f29f7c311020817e70d4de3af96ca71
+  data.tar.gz: 939a492793d040c4555e73eedf6dc8a66cc4c72121addfdc3ac659ba5642599ad6960c1a3182299e0e98321d7dec52e524b52450d11d0eb5766e546e612a387c

data/app/jobs/foreman_webhooks/deliver_webhook_job.rb

@@ -6,13 +6,25 @@ module ForemanWebhooks
 
     def perform(options)
       webhook = Webhook.unscoped.find_by(id: options[:webhook_id])
-      WebhookService.new(
+      result = WebhookService.new(
         webhook: webhook,
         headers: options[:headers],
         url: options[:url],
         event_name: options[:event_name],
         payload: options[:payload]
       ).execute
+
+      return unless result[:status] == :error
+
+      raise [result[:http_status], result[:message]].compact.join(': ')
+    end
+
+    rescue_from(StandardError) do |error|
+      Foreman::Logging.logger('background').error(
+        'DeliverWebhook: '\
+        "Error while delivering - #{error.message}"
+      )
+      raise error # propagate the error to the tasking system to properly report it there
     end
 
     def webhook_id

data/app/models/webhook.rb

@@ -82,8 +82,8 @@ class Webhook < ApplicationRecord
   def test(payload: nil)
     ForemanWebhooks::WebhookService.new(
       webhook: self,
-      headers: rendered_headers(event, {}),
-      url: rendered_targed_url(event, {}),
+      headers: http_headers,
+      url: target_url,
       event_name: event,
       payload: test_payload(payload || '')
     ).execute

data/app/views/foreman_webhooks/webhook_templates/katello_-_content_view.erb

@@ -1,6 +1,6 @@
 <%#
 name: Katello Content View
-description: Example payload for content_view_*.event.foreman
+description: Example payload for actions.katello.content_view_*.event.foreman
 snippet: false
 model: WebhookTemplate
 -%>

data/app/views/foreman_webhooks/webhook_templates/katello_-_promote.erb

@@ -1,6 +1,6 @@
 <%#
 name: Katello Promote
-description: Example payload for actions.katello.content_view.promote_suceeded
+description: Example payload for actions.katello.content_view.promote_suceeded.event.foreman
 snippet: false
 model: WebhookTemplate
 -%>

data/app/views/foreman_webhooks/webhook_templates/katello_-_publish.erb

@@ -1,6 +1,6 @@
 <%#
 name: Katello Publish
-description: Example payload for actions.katello.content_view.publish_suceeded
+description: Example payload for actions.katello.content_view.publish_suceeded.event.foreman
 snippet: false
 model: WebhookTemplate
 -%>

data/app/views/foreman_webhooks/webhook_templates/katello_-_repo_sync.erb

@@ -1,6 +1,6 @@
 <%#
 name: Katello Sync
-description: Example payload for actions.katello.repository.sync_succeeded
+description: Example payload for actions.katello.repository.sync_succeeded.event.foreman
 snippet: false
 model: WebhookTemplate
 -%>

data/app/views/foreman_webhooks/webhook_templates/remote_execution_-_host_job.erb

@@ -1,6 +1,6 @@
 <%#
 name: Remote Execution Host Job
-description: Example payload for actions.remote_execution.run_host_job_succeeded
+description: Example payload for actions.remote_execution.run_host_job_succeeded.event.foreman
 snippet: false
 model: WebhookTemplate
 -%>

data/lib/foreman_webhooks/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ForemanWebhooks
-  VERSION = "3.2.1"
+  VERSION = "3.2.2"
 end

data/test/jobs/foreman_webhooks/deliver_webhook_job_test.rb

@@ -10,7 +10,8 @@ module ForemanWebhooks
     end
 
     it 'executes the webhook service' do
-      ::ForemanWebhooks::WebhookService.any_instance.expects(:execute).once
+      result = { status: :success }
+      ::ForemanWebhooks::WebhookService.any_instance.expects(:execute).once.returns(result)
       job.perform_now
     end
   end

data/webpack/ForemanWebhooks/Routes/Webhooks/WebhooksIndexPage/Components/WebhookTestModal.js

@@ -58,18 +58,16 @@ const WebhookTestModal = ({ toTest }) => {
       enforceFocus
       className="webhooks-modal"
     >
-      <p>
-        {sprintf(
-          __(
-            'You are about to test %s webhook.' +
-              '\n' +
-              'Please, note that this will not contain actual information or render the attached template.' +
-              '\n' +
-              'You can specify below a custom payload to test the webhook with.'
-          ),
-          name
-        )}
-      </p>
+      {`${sprintf(__('You are about to test %s webhook.'), name)} `}
+      {`${__(
+        'Please, note that this will not contain actual information or render the attached template.'
+      )} `}
+      {`${__(
+        'In case you are using dynamic URL (ERB template), this will not be rendered correctly due to the absence of a real object.'
+      )} `}
+      {__('You can specify below a custom payload to test the webhook with.')}
+      <br />
+      <br />
       <ForemanForm
         onSubmit={handleSubmit}
         initialValues={initialTestValues}

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: foreman_webhooks
 version: !ruby/object:Gem::Version
-  version: 3.2.1
+  version: 3.2.2
 platform: ruby
 authors:
 - Timo Goebel
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-08-09 00:00:00.000000000 Z
+date: 2023-11-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -245,7 +245,7 @@ licenses:
 - GPL-3.0
 metadata:
   is_foreman_plugin: 'true'
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -260,8 +260,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
-signing_key: 
+rubygems_version: 3.4.10
+signing_key:
 specification_version: 4
 summary: Configure webhooks for Foreman.
 test_files: