foreman_webhooks 3.0.4 → 3.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5382cbf96f38f9ab42cef651aaf8eb1e602d2c651352c370138b911e7cd10901
-  data.tar.gz: 5e32f69db8731876e51af931c9d3b174c9dcae62ae3ebdb11b186d70b5fff384
+  metadata.gz: 546a669c4f3c26a491fb5ab87ce068aa5391085cd5e0994e793d4f2d56c19a63
+  data.tar.gz: 5e19ad3a87e48dd0759cd7c418fadec7cd1574dd157b4e15563e43f1a3bce3e9
 SHA512:
-  metadata.gz: fc5314756061144ef29988e98b7344c25b2df0e13b3cb122a965d071c18c5fdb9e46badc061b50f2e79b76e33ebf95e8d9c54671bac1dd0e46b334e070a9048a
-  data.tar.gz: f9e3824a1b66dfdd988eeb9b3bbbd922d9e9b460bb7922a0139130a47cf7074ed958449c82e97941b7492328b361f2296026168b41ee85d492689ae691620012
+  metadata.gz: cba01a2168b1916b5874a8b39d50cd4cc463955e63352a84d8794183078e898f301eec2b79a893745caa1c7ad3f9d493a8b1363a8d0eeac53a2fa5d740bc3722
+  data.tar.gz: fd537f4c7dd5e995ad5f5f2116a2c0670d2df8bbe1398cc982684eee8fbb558dae9ef36b939cd557a7ea6d57d455ed12a3579ff7ec05cede892d73b92c145193

data/app/views/api/v2/webhooks/show.json.rabl

@@ -15,6 +15,10 @@ attributes :target_url,
            :ssl_ca_certs,
            :user
 
+node :password_set do |webhook|
+  webhook.password.present?
+end
+
 child :webhook_template do
   extends 'api/v2/webhook_templates/base'
 end

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

@@ -20,4 +20,4 @@ model: WebhookTemplate
 # Task ended at <%= @object.task.ended_at %>
 # Task resulted with <%= @object.task.result %>
 # Task state <%= @object.task.state %>
-# Task action output <%= @object.task.action_output %>
+# Task action output <%= @object.task.action_continuous_output %>

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

@@ -21,4 +21,4 @@ model: WebhookTemplate
 # Task ended at <%= @object.task.ended_at %>
 # Task resulted with <%= @object.task.result %>
 # Task state <%= @object.task.state %>
-# Task action output <%= @object.task.action_output %>
+# Task action output <%= @object.task.action_continuous_output %>

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

@@ -24,4 +24,4 @@ model: WebhookTemplate
 # Task ended at <%= @object.task.ended_at %>
 # Task resulted with <%= @object.task.result %>
 # Task state <%= @object.task.state %>
-# Task action output <%= @object.task.action_output %>
+# Task action output <%= @object.task.action_continuous_output %>

data/lib/foreman_webhooks/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ForemanWebhooks
-  VERSION = '3.0.4'
+  VERSION = "3.1.0"
 end

data/webpack/ForemanWebhooks/Routes/Webhooks/Components/WebhookForm/Components/ForemanFormikField.js

@@ -21,6 +21,8 @@ const ForemanFormikField = ({
   placeholder,
   options,
   isLoading,
+  disabled,
+  setDisabled,
 }) => (
   <FormikField name={name}>
     {({
@@ -32,6 +34,15 @@ const ForemanFormikField = ({
 
         return filter(initialOptions, o => o.value === fieldValue);
       };
+      const passwordInput = (
+        <input
+          {...field}
+          placeholder={disabled ? '********' : ''}
+          type={type}
+          disabled={disabled}
+          className="form-control"
+        />
+      );
       let content = null;
       switch (type) {
         case 'textarea':
@@ -56,10 +67,30 @@ const ForemanFormikField = ({
               onChange={selected =>
                 setFieldValue(field.name, selected[0]?.value)
               }
-              // onBlur={e => setFieldTouched(field.name, true)}
             />
           );
           break;
+        case 'password':
+          content = setDisabled ? (
+            <div className="input-group">
+              {passwordInput}
+              <span className="input-group-btn">
+                <button
+                  className="btn btn-default"
+                  onClick={e => {
+                    e.preventDefault();
+                    setDisabled(!disabled);
+                  }}
+                  title={__('Change the password')}
+                >
+                  <span className="pficon pficon-edit" />
+                </button>
+              </span>
+            </div>
+          ) : (
+            passwordInput
+          );
+          break;
         default:
           content = (
             <input
@@ -101,6 +132,8 @@ ForemanFormikField.propTypes = {
   placeholder: PropTypes.string,
   options: PropTypes.array,
   isLoading: PropTypes.bool,
+  disabled: PropTypes.bool,
+  setDisabled: PropTypes.func,
 };
 
 ForemanFormikField.defaultProps = {
@@ -112,6 +145,8 @@ ForemanFormikField.defaultProps = {
   placeholder: '',
   options: null,
   isLoading: false,
+  disabled: false,
+  setDisabled: undefined,
 };
 
 export default ForemanFormikField;

data/webpack/ForemanWebhooks/Routes/Webhooks/Components/WebhookForm/Components/WebhookFormTabs.js

@@ -18,6 +18,8 @@ const WebhookFormTabs = ({
   availableEvents,
   isTemplatesLoading,
   isEventsLoading,
+  isPasswordDisabled,
+  setIsPasswordDisabled,
 }) => (
   <Tabs activeKey={activeTab} onSelect={handleTabClick} isFilled>
     <Tab
@@ -90,6 +92,8 @@ const WebhookFormTabs = ({
           type="password"
           label={__('Password')}
           labelHelp={__('Authentication credentials')}
+          disabled={isPasswordDisabled}
+          setDisabled={setIsPasswordDisabled}
         />
         <ForemanFormikField
           name="verify_ssl"
@@ -153,11 +157,15 @@ WebhookFormTabs.propTypes = {
   availableEvents: PropTypes.array.isRequired,
   isTemplatesLoading: PropTypes.bool.isRequired,
   isEventsLoading: PropTypes.bool.isRequired,
+  isPasswordDisabled: PropTypes.bool,
+  setIsPasswordDisabled: PropTypes.func,
 };
 
 WebhookFormTabs.defaultProps = {
   disabled: false,
   formProps: {},
+  isPasswordDisabled: false,
+  setIsPasswordDisabled: undefined,
 };
 
 export default WebhookFormTabs;

data/webpack/ForemanWebhooks/Routes/Webhooks/Components/WebhookForm/WebhookForm.js

@@ -25,6 +25,8 @@ const WebhookForm = ({
   availableEvents,
   isTemplatesLoading,
   isEventsLoading,
+  isPasswordDisabled,
+  setIsPasswordDisabled,
 }) => {
   const webhookTemplates = templates.map(t => ({ value: t.id, label: t.name }));
 
@@ -51,6 +53,8 @@ const WebhookForm = ({
         availableEvents={availableEvents}
         isEventsLoading={isEventsLoading}
         isTemplatesLoading={isTemplatesLoading}
+        isPasswordDisabled={isPasswordDisabled}
+        setIsPasswordDisabled={setIsPasswordDisabled}
       />
     </ForemanForm>
   );
@@ -64,6 +68,13 @@ WebhookForm.propTypes = {
   availableEvents: PropTypes.array.isRequired,
   isEventsLoading: PropTypes.bool.isRequired,
   isTemplatesLoading: PropTypes.bool.isRequired,
+  isPasswordDisabled: PropTypes.bool,
+  setIsPasswordDisabled: PropTypes.func,
+};
+
+WebhookForm.defaultProps = {
+  isPasswordDisabled: false,
+  setIsPasswordDisabled: undefined,
 };
 
 export default WebhookForm;

data/webpack/ForemanWebhooks/Routes/Webhooks/Components/WebhookForm/__tests__/__snapshots__/WebhookForm.test.js.snap

@@ -241,6 +241,7 @@ exports[`WebhookForm rendering should render for edit page 1`] = `
       ]
     }
     isEventsLoading={false}
+    isPasswordDisabled={false}
     isTemplatesLoading={false}
     webhookTemplates={
       Array [
@@ -492,6 +493,7 @@ exports[`WebhookForm rendering should render for new page 1`] = `
       ]
     }
     isEventsLoading={false}
+    isPasswordDisabled={false}
     isTemplatesLoading={false}
     webhookTemplates={
       Array [

data/webpack/ForemanWebhooks/Routes/Webhooks/Components/WebhookForm/index.js

@@ -24,7 +24,13 @@ import {
 
 const params = { page: 1, search: 'snippet = false', per_page: 'all' };
 
-const ConnectedWebhookForm = ({ onCancel, handleSubmit, initialValues }) => {
+const ConnectedWebhookForm = ({
+  onCancel,
+  handleSubmit,
+  initialValues,
+  isPasswordDisabled,
+  setIsPasswordDisabled,
+}) => {
   const dispatch = useDispatch();
 
   const templates = useSelector(selectWebhookTemplates);
@@ -60,6 +66,8 @@ const ConnectedWebhookForm = ({ onCancel, handleSubmit, initialValues }) => {
       initialValues={initialValues}
       isTemplatesLoading={isTemplatesLoading}
       isEventsLoading={isEventsLoading}
+      isPasswordDisabled={isPasswordDisabled}
+      setIsPasswordDisabled={setIsPasswordDisabled}
     />
   );
 };
@@ -68,6 +76,13 @@ ConnectedWebhookForm.propTypes = {
   onCancel: PropTypes.func.isRequired,
   handleSubmit: PropTypes.func.isRequired,
   initialValues: PropTypes.object.isRequired,
+  isPasswordDisabled: PropTypes.bool,
+  setIsPasswordDisabled: PropTypes.func,
+};
+
+ConnectedWebhookForm.defaultProps = {
+  isPasswordDisabled: false,
+  setIsPasswordDisabled: undefined,
 };
 
 export default ConnectedWebhookForm;

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

@@ -1,5 +1,6 @@
 import React from 'react';
 import PropTypes from 'prop-types';
+import { Modal } from 'patternfly-react';
 import { useDispatch } from 'react-redux';
 
 import { translate as __ } from 'foremanReact/common/I18n';
@@ -47,11 +48,12 @@ const WebhookCreateModal = ({ onSuccess, onCancel }) => {
   return (
     <ForemanModal
       id={WEBHOOK_CREATE_MODAL_ID}
-      title={__('Create Webhook')}
       backdrop="static"
       className="webhooks-modal"
     >
-      <ForemanModal.Header />
+      <Modal.Header>
+        <Modal.Title>{__('Create Webhook')}</Modal.Title>
+      </Modal.Header>
       <ConnectedWebhookForm
         handleSubmit={handleSubmit}
         initialValues={initialWebhookValues}

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

@@ -1,4 +1,5 @@
-import React, { useEffect } from 'react';
+import React, { useEffect, useState } from 'react';
+import { Modal } from 'patternfly-react';
 import { useSelector, useDispatch } from 'react-redux';
 import PropTypes from 'prop-types';
 
@@ -29,22 +30,11 @@ import './WebhookModal.scss';
 const WebhookEditModal = ({ toEdit, onSuccess, onCancel }) => {
   const dispatch = useDispatch();
 
+  const [isPasswordDisabled, setIsPasswordDisabled] = useState(false);
   const id = toEdit;
 
-  const handleSubmit = (values, actions) =>
-    dispatch(
-      submitForm({
-        url: foremanUrl(`/api${WEBHOOKS_PATH}/${id}`),
-        values: { ...values, controller: 'webhooks' },
-        item: 'Webhook',
-        message: __('Webhook was successfully updated.'),
-        method: 'put',
-        successCallback: onSuccess,
-        actions,
-      })
-    );
-
   const isLoading = useSelector(selectIsLoading);
+  const isPasswordSet = useSelector(selectWebhookValues).passwordSet;
   const initialWebhookValues = {
     id: useSelector(selectWebhookValues).id,
     name: useSelector(selectWebhookValues).name,
@@ -62,6 +52,27 @@ const WebhookEditModal = ({ toEdit, onSuccess, onCancel }) => {
     proxy_authorization: useSelector(selectWebhookValues).proxyAuthorization,
   };
 
+  useEffect(() => {
+    setIsPasswordDisabled(isPasswordSet);
+  }, [isPasswordSet]);
+
+  const handleSubmit = (values, actions) => {
+    if (isPasswordDisabled) {
+      delete values.password;
+    }
+    dispatch(
+      submitForm({
+        url: foremanUrl(`/api${WEBHOOKS_PATH}/${id}`),
+        values: { ...values, controller: 'webhooks' },
+        item: 'Webhook',
+        message: __('Webhook was successfully updated.'),
+        method: 'put',
+        successCallback: onSuccess,
+        actions,
+      })
+    );
+  };
+
   useEffect(() => {
     if (id) {
       dispatch(
@@ -73,21 +84,31 @@ const WebhookEditModal = ({ toEdit, onSuccess, onCancel }) => {
     }
   }, [id, dispatch]);
 
+  const onEditCancel = () => {
+    if (isPasswordSet) setIsPasswordDisabled(true);
+    onCancel();
+  };
+
   return (
     <ForemanModal
       id={WEBHOOK_EDIT_MODAL_ID}
-      title={`${__('Edit')} ${initialWebhookValues.name}`}
       backdrop="static"
       className="webhooks-modal"
     >
-      <ForemanModal.Header />
+      <Modal.Header>
+        <Modal.Title>
+          {`${__('Edit')} ${initialWebhookValues.name}`}
+        </Modal.Title>
+      </Modal.Header>
       {isLoading ? (
         <Loading />
       ) : (
         <ConnectedWebhookForm
           handleSubmit={handleSubmit}
           initialValues={initialWebhookValues}
-          onCancel={onCancel}
+          onCancel={onEditCancel}
+          isPasswordDisabled={isPasswordDisabled}
+          setIsPasswordDisabled={setIsPasswordDisabled}
         />
       )}
     </ForemanModal>

metadata

@@ -1,15 +1,43 @@
 --- !ruby/object:Gem::Specification
 name: foreman_webhooks
 version: !ruby/object:Gem::Version
-  version: 3.0.4
+  version: 3.1.0
 platform: ruby
 authors:
 - Timo Goebel
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-08-16 00:00:00.000000000 Z
-dependencies: []
+date: 2023-03-21 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rdoc
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Plugin for Foreman that allows to configure Webhooks.
 email:
 - mail@timogoebel.name
@@ -175,7 +203,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.1.6
 signing_key: 
 specification_version: 4
 summary: Configure webhooks for Foreman.