foreman_openbolt 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 19c30e74d249eacf814179dac70292c3e42182b39477cab06adf304885298587
-  data.tar.gz: bbc28cd01be2a7f100d1ec015c92aff471584e7445468184e7ef1bd97c485b27
+  metadata.gz: b212e4874311ac07caa39290021ef940849d6a64c4e903f393497b2cf97654a5
+  data.tar.gz: e8658c692b134da9b542234b30064f6274bd0b1aed2ad4f25ef5397b015afeb8
 SHA512:
-  metadata.gz: 2b40018885e22e7d3690b13c524787dadbf8ad2fe8fa5af2e82e5eb4fd99d65a56a7015cbd02717573c5cd12395d1a157130c16257fa94a5b3f28ef78470fd55
-  data.tar.gz: 753476abd475f3de15ad928874048c9772d4a55534129f35b9d4c032beb5c0f787e8ba6aed271763bb59d66383986360cbf8d0438618aefdb9f2175790b959c7
+  metadata.gz: 8fcfe54eef80b1951fb8cc483659b6219ede35010b120670b2dab5de998787bcc965bc3b73756d2ba7c9a684cc75a4fb007c7bc85a619f42168d63982029f275
+  data.tar.gz: dac7ac671cfa773d0ec2a28fcb0562634b59a38d7eed754f3fde79172f3c3da381f70a9ed3ef8a9f19fc34fc0d033931a2fdd2fabc302f88e6208c40029b72cc

data/README.md

@@ -23,14 +23,14 @@ A task is copied to N targets and executed there.
 
 ## Plans
 
-Plans provide complex logic options, written in Puppet DSL.
-Besides the usual Puppet DSL functions, it's also possible to execute tasks and evaluate their responses.
+Plans provide complex logic options, written in the Puppet language.
+Besides the usual Puppet language functions, it's also possible to execute tasks and evaluate their responses.
 
 ## OpenBolt in Foreman!
 
-OpenBolt is the Ansible counterpart and OpenBolt is Puppet "native".
-OpenBolt and Puppet integrate very well together and OpenBolt can reuse your existing Puppet code.
-Since OpenBolt is a CLI only application, and most Puppet users run Foreman anyways, it made sense to integrate OpenBolt into Foreman, instead of writing another web UI.
+OpenBolt is the Ansible counterpart and OpenBolt is OpenVox/Puppet native.
+OpenBolt and OpenVox/Puppet integrate very well together and OpenBolt can reuse your existing code.
+Since OpenBolt is a CLI only application, and most OpenVox and Puppet users run Foreman anyways, it made sense to integrate OpenBolt into Foreman, instead of writing another web UI.
 
 ## Installation
 
@@ -49,7 +49,9 @@ The [theforeman/foreman_proxy](https://github.com/theforeman/puppet-foreman_prox
 The Foreman plugin provides UI elements to start Tasks on various nodes.
 Foreman then talks to a Smartproxy to run OpenBolt.
 The Smartproxy also establishes the connections to the various targets.
-This is usually a ssh or WinRM connection (and soon choria, see [the TODO section](#todo)).
+This is usually a SSH, WinRM, or Choria connection. The Choria transport
+requires OpenBolt 5.5 or later and is not available in Puppet Bolt.
+SSH and WinRM transports work with any version.
 
 You need to have `bolt` in your `$PATH` on the Smartproxy.
 OpenBolt packages are available at [yum.voxpupuli.org](https://yum.voxpupuli.org/) & [apt.voxpupuli.org](https://apt.voxpupuli.org/) in the openvox8 repo.
@@ -57,172 +59,18 @@ You can also use the legacy Bolt packages from Perforce from the `puppet-tools`
 
 The integration is supported on Foreman 3.17 and all following versions, including development/nightly builds.
 
-OpenBolt relies on Tasks & Plans. They are distributed as puppet modules.
+OpenBolt relies on Tasks & Plans. They are distributed as modules.
 The plugin assumes that you deployed your code.
 We recommend to use [r10k](https://github.com/puppetlabs/r10k?tab=readme-ov-file#r10k) or [g10k](https://github.com/xorpaul/g10k?tab=readme-ov-file#g10k) to deploy code, as you do it on your compilers.
 
 A handful of core/default Tasks & Plans are also included in the [OpenBolt rpm/deb packages](https://github.com/OpenVoxProject/openbolt/blob/main/Puppetfile).
 
-## Usage
+## Documentation
 
-(all screenshots were taken on Foreman 3.17)
-
-After installation, you will see a new UI element
-
-![foreman UI menu screenshot](./ext/foreman-ui-menu.png)
-
-The "Launch Task" option allows you to select any smartproxy with the `openbolt` feature (which is available when the OpenBolt Smartproxy plugin is installed).
-Afterwards you can select N targets to run the task and select an available task from the selected Smartproxy.
-On the right side you can configure OpenBolt connection settings.
-
-![launch task detail view](./ext/foreman-launch-task.png)
-
-After selecting a task, the task metadata is fetched and shown.
-Additional input elements will appear, if the task support it.
-
-![service task metadata](./ext/task-metadata-minimal.png)
-
-The metadata can contains a description and datatypes for tasks.
-Those information can be shown as well.
-
-![service task detailed metadata](./ext/task-metadata.png)
-
-While the task is running, the UI polls the status from the smart proxy.
-
-![task loading screen](./ext/task-running.png)
-
-After the task finished, it will display a success for failure page.
-
-![failed task view](./ext/task-execution-details.png)
-
-You can also see the used parameters for a task.
-
-![task used parameters](./ext/task-task-details.png)
-
-We also display the used OpenBolt command line, in case you want to manually run it or debug it.
-
-![display used OpenBolt command](./ext/task-log-output.png)
-
-OpenBolt returns JSON for executed tasks.
-That's visible in the UI.
-For failed tasks but also for passed tasks.
-
-![failed task output](./ext/task-result.png)
-
-![service task passed on two nodes](./ext/task-successful-result.png)
-
-## Development
-
-### Linting
-
-```bash
-bundle exec rake lint        # Run all linters (rubocop, erb_lint, eslint)
-bundle exec rake lint:fix    # Auto-fix where possible
-```
-
-Ruby and ERB linters run directly. The JavaScript linter requires npm dependencies, so either install them locally (`npm install --legacy-peer-deps`) or run lint:js inside a container:
-
-```bash
-CONTAINER=1 bundle exec rake lint:js
-```
-
-### Unit Tests
-
-Unit tests run inside Docker containers with a full Foreman installation. Requires Docker with compose support.
-
-```bash
-bundle exec rake test:unit:up    # Build image, start containers, install deps
-bundle exec rake test:unit:ruby  # Run Ruby tests
-bundle exec rake test:unit:js    # Run JavaScript tests
-bundle exec rake test:unit:all   # Run all unit tests
-bundle exec rake test:unit:down  # Stop and remove containers
-bundle exec rake test              # Shortcut: up, test, down in one step
-```
-
-Set `FOREMAN_VERSION` to test against a specific Foreman version (default: `3.18`):
-
-```bash
-FOREMAN_VERSION=3.17 bundle exec rake test:unit:up
-```
-
-### Acceptance Tests
-
-Acceptance tests exercise the plugin through the browser using Capybara and Selenium. They build RPMs, start a multi-container environment (Foreman + OpenVox + SSH targets + Chromium), and run tests against the real UI.
-
-**Prerequisites:**
-
-```bash
-bundle install --with acceptance
-```
-
-The [smart_proxy_openbolt](https://github.com/overlookinfra/smart_proxy_openbolt) and [foreman-packaging](https://github.com/theforeman/foreman-packaging) repos are cloned automatically when needed.
-
-**Running:**
-
-```bash
-bundle exec rake acceptance         # Full cycle: up, run tests, down
-bundle exec rake acceptance:up      # Build RPMs, start Foreman, configure everything
-bundle exec rake acceptance:run     # Run tests (requires up first)
-bundle exec rake acceptance:down    # Stop containers
-bundle exec rake acceptance:clean   # Full reset: stop containers, remove images and artifacts
-```
-
-The `acceptance:up` task is idempotent and can be re-run to pick up new RPM changes. It caches the Foreman Docker image per version so subsequent runs are faster.
-
-**Watching tests in the browser:**
-
-Set `HEADFUL=1` to disable headless mode, then open `http://localhost:7900` (password: `secret`) to watch the tests via noVNC:
-
-```bash
-HEADFUL=1 bundle exec rake acceptance:run
-```
-
-**Running a subset of tests:**
-
-`acceptance:run` accepts `TEST=<path>` to limit which test files are loaded, and `TESTOPTS=<opts>` to forward options (e.g. `--name=/pattern/`) to the Test::Unit autorunner. Both can be combined.
-
-```bash
-# Run every test in one file
-bundle exec rake acceptance:run TEST=test/acceptance/tests/settings_test.rb
-
-# Run a single test by exact method name (any file)
-bundle exec rake acceptance:run TESTOPTS='--name=test_echo_task_succeeds_on_all_targets'
-
-# Run tests whose name matches a regex within one file
-bundle exec rake acceptance:run \
-  TEST=test/acceptance/tests/settings_test.rb \
-  TESTOPTS='--name=/host_key/'
-```
-
-**Environment variables:**
-
-| Variable | Default | Description |
-|----------|---------|-------------|
-| `CHROMEDRIVER_URL` | `http://localhost:4444` | Selenium WebDriver endpoint |
-| `FOREMAN_BRANCH` | `<version>-stable` | Foreman git branch for unit test image (derived from `FOREMAN_VERSION`) |
-| `FOREMAN_PACKAGING_REPO` | `https://github.com/theforeman/foreman-packaging.git` | Git URL for foreman-packaging (cloned automatically for RPM builds) |
-| `FOREMAN_PASS` | `changeme` | Foreman login password |
-| `FOREMAN_URL` | `https://foreman` | Foreman URL as seen by Chrome. Override to run tests against a live instance |
-| `FOREMAN_USER` | `admin` | Foreman login username |
-| `FOREMAN_VERSION` | `3.18` | Foreman version to test against |
-| `HEADFUL` | unset | Set to `1` to show the browser in noVNC |
-| `SELENIUM_IMAGE` | auto-detected (ARM/x86) | Selenium container image (auto-selects `seleniarm/standalone-chromium` or `selenium/standalone-chrome`) |
-| `SMART_PROXY_OPENBOLT_REF` | `main` | Branch or tag to clone |
-| `SMART_PROXY_OPENBOLT_REPO` | `https://github.com/overlookinfra/smart_proxy_openbolt.git` | Git URL for smart_proxy_openbolt (cloned automatically for RPM builds) |
-
-### Building Packages
-
-Build RPM or DEB packages locally using containers. The [foreman-packaging](https://github.com/theforeman/foreman-packaging) repo is cloned automatically:
-
-```bash
-bundle exec rake build:rpm   # Build RPM
-bundle exec rake build:deb   # Build DEB
-```
-
-## TODO
-
-* Integrate plans into the web UI
-* Provide a choria transport plugin
+- [Usage](docs/usage.md): screenshots and walkthrough of the UI
+- [Development](docs/development.md): linting, unit tests, acceptance tests, building packages
+- [Releasing](docs/releasing.md): version locations, release steps, RPM/DEB packaging
+- [Choria Testing](docs/choria-testing.md): setting up Choria on a Foreman install for transport testing
 
 ## Contributing & support
 
@@ -247,89 +95,3 @@ GNU General Public License for more details.
 
 You should have received a copy of the GNU General Public License
 along with this program.  If not, see <http://www.gnu.org/licenses/>.
-
-## How to Release
-
-### Version locations
-
-Update the version in these files:
-
-1. `lib/foreman_openbolt/version.rb` -- the gem version (authoritative source)
-2. `package.json` -- the npm package version (must match)
-
-If the minimum Foreman version changes, also update:
-
-3. `lib/foreman_openbolt/engine.rb` -- `requires_foreman '>= X.Y.Z'`
-4. `foreman_openbolt.spec.erb` -- `%global foreman_min_version X.Y.Z`
-5. `.github/workflows/build.yml` -- default `foreman_version` and `foreman_packaging_ref` inputs
-
-### Release steps
-
-1. Bump the version in the two files listed above
-2. Generate the changelog:
-   ```bash
-   CHANGELOG_GITHUB_TOKEN=github_pat_... bundle exec rake changelog
-   ```
-3. Create a PR with the version bump and changelog, get it reviewed and merged
-4. Create and push a tag matching the version:
-   ```bash
-   git tag 1.1.0
-   git push origin 1.1.0
-   ```
-5. The [release workflow](.github/workflows/release.yml) runs automatically on tag push and:
-   - Builds the gem
-   - Creates a GitHub Release with auto-generated notes and the gem attached
-   - Publishes the gem to GitHub Packages
-   - Publishes the gem to RubyGems.org (requires the `release` environment)
-   - Verifies the gem is available on RubyGems.org
-
-### RPM/DEB packaging
-
-After the gem is published to RubyGems, both RPM and DEB packages need to be updated in [theforeman/foreman-packaging](https://github.com/theforeman/foreman-packaging).
-
-A bot automatically creates PRs against the `rpm/develop` and `deb/develop` branches to pick up the new gem version. These PRs build packages for Foreman nightly.
-
-For stable Foreman releases (currently 3.17 and 3.18), cherry-pick the packaging commits from the develop branches into the corresponding stable branches. For each stable version you want to support:
-
-```bash
-cd foreman-packaging
-
-# RPM: cherry-pick from rpm/develop into a branch off the stable target
-git checkout rpm/3.18
-git checkout -b cherry-pick/rubygem-foreman_openbolt-rpm-3.18
-git cherry-pick <commit-from-rpm/develop>
-# Push to your fork and open a PR targeting rpm/3.18
-
-# DEB: same approach for the deb side
-git checkout deb/3.18
-git checkout -b cherry-pick/rubygem-foreman-openbolt-deb-3.18
-git cherry-pick <commit-from-deb/develop>
-# Push to your fork and open a PR targeting deb/3.18
-```
-
-PRs against stable branches should be labeled "Stable branch".
-
-**Alternative: manual version bump**
-
-If the cherry-pick doesn't apply cleanly, you can bump the version manually on the stable branch instead.
-
-*RPM:* Checkout the target branch and run `bump_rpm.sh`:
-```bash
-cd foreman-packaging
-git checkout rpm/3.18
-git checkout -b bump_rpm/rubygem-foreman_openbolt
-./bump_rpm.sh packages/plugins/rubygem-foreman_openbolt
-# Review changes, push to your fork, and open a PR targeting rpm/3.18
-```
-
-*DEB:* Checkout the target branch and update these files:
-- `debian/gem.list` -- new gem filename
-- `foreman_openbolt.rb` -- new version
-- `debian/control` -- dependency versions (if changed)
-- `debian/changelog` -- add a new entry
-
-```bash
-git checkout deb/3.18
-git checkout -b bump_deb/ruby-foreman-openbolt
-# Make the changes above, push to your fork, and open a PR targeting deb/3.18
-```

data/app/controllers/foreman_openbolt/task_controller.rb

@@ -53,7 +53,7 @@ module ForemanOpenbolt
         key = setting.name.sub(/^openbolt_/, '')
         if setting.encrypted?
           defaults[key] = ENCRYPTED_PLACEHOLDER unless setting.value.to_s.empty?
-        else
+        elsif !setting.value.to_s.empty?
           defaults[key] = setting.value
         end
       end
@@ -136,6 +136,9 @@ module ForemanOpenbolt
       rescue ActiveRecord::RecordInvalid => e
         log_exception('launch_task', e)
         render_error("Database error: #{e.message}", :internal_server_error)
+      rescue ProxyAPI::ProxyException => e
+        log_exception('launch_task', e)
+        render_error("Smart Proxy error: #{e.message}", :bad_gateway)
       rescue StandardError => e
         log_exception('launch_task', e)
         render_error("Error launching task: #{e.message}", :internal_server_error)

data/lib/foreman_openbolt/engine.rb

@@ -35,6 +35,7 @@ module ForemanOpenbolt
               TRANSPORTS = {
                 ssh: N_('SSH'),
                 winrm: N_('WinRM'),
+                choria: N_('Choria'),
               }.freeze
               LOG_LEVELS = {
                 error: N_('Error'),
@@ -43,28 +44,19 @@ module ForemanOpenbolt
                 debug: N_('Debug'),
                 trace: N_('Trace'),
               }.freeze
+              CHORIA_TASK_AGENTS = {
+                bolt_tasks: N_('Bolt Tasks'),
+                shell: N_('Shell'),
+              }.freeze
               # rubocop:enable Lint/ConstantDefinitionInBlock
 
-              setting 'openbolt_transport',
-                type: :string,
-                default: 'ssh',
-                full_name: N_('Transport'),
-                description: N_('The transport method to use for connecting to target hosts'),
-                collection: proc { TRANSPORTS }
+              # General (all transports)
               setting 'openbolt_log-level',
                 type: :string,
                 default: 'debug',
                 full_name: N_('Log Level'),
                 description: N_('Set the log level during OpenBolt execution'),
                 collection: proc { LOG_LEVELS }
-              setting 'openbolt_verbose',
-                type: :boolean,
-                default: false,
-                full_name: N_('Verbose'),
-                description: N_(
-                  'Run the OpenBolt command with the --verbose flag. This prints additional information ' \
-                  'during OpenBolt execution and will print any out::verbose plan statements.'
-                )
               setting 'openbolt_noop',
                 type: :boolean,
                 default: false,
@@ -72,22 +64,132 @@ module ForemanOpenbolt
                 description: N_(
                   'Run the OpenBolt command with the --noop flag, which will make no changes to the target host'
                 )
+              setting 'openbolt_password',
+                type: :string,
+                default: nil,
+                full_name: N_('Password'),
+                description: N_('Password used for SSH or WinRM authentication'),
+                encrypted: true
               setting 'openbolt_tmpdir',
                 type: :string,
-                default: '',
+                default: nil,
                 full_name: N_('Temporary Directory'),
                 description: N_('Directory to use for temporary files on target hosts during OpenBolt execution')
+              setting 'openbolt_transport',
+                type: :string,
+                default: 'ssh',
+                full_name: N_('Transport'),
+                description: N_('The transport method to use for connecting to target hosts'),
+                collection: proc { TRANSPORTS }
               setting 'openbolt_user',
                 type: :string,
-                default: '',
+                default: nil,
                 full_name: N_('User'),
                 description: N_('Username used for SSH or WinRM authentication')
-              setting 'openbolt_password',
+              setting 'openbolt_verbose',
+                type: :boolean,
+                default: false,
+                full_name: N_('Verbose'),
+                description: N_(
+                  'Run the OpenBolt command with the --verbose flag. This prints additional information ' \
+                  'during OpenBolt execution and will print any out::verbose plan statements.'
+                )
+
+              # Choria
+              setting 'openbolt_choria-broker-timeout',
+                type: :integer,
+                default: nil,
+                full_name: N_('Choria Broker Timeout'),
+                description: N_('Time in seconds to wait when establishing a connection to a Choria broker.')
+              setting 'openbolt_choria-brokers',
                 type: :string,
-                default: '',
-                full_name: N_('Password'),
-                description: N_('Password used for SSH or WinRM authentication'),
-                encrypted: true
+                default: nil,
+                full_name: N_('Choria Brokers'),
+                description: N_(
+                  'Comma-separated list of Choria broker addresses in host or host:port format ' \
+                  '(e.g. broker1.example.com:4222,broker2.example.com:4222). Port defaults to 4222 if omitted. ' \
+                  'When not set, the Choria client checks the config file, then SRV records, then falls back to puppet:4222.'
+                )
+              setting 'openbolt_choria-collective',
+                type: :string,
+                default: nil,
+                full_name: N_('Choria Collective'),
+                description: N_('Choria collective to route messages through.')
+              setting 'openbolt_choria-command-timeout',
+                type: :integer,
+                default: nil,
+                full_name: N_('Choria Command Timeout'),
+                description: N_('Time in seconds to wait for command completion on target nodes.')
+              setting 'openbolt_choria-config-file',
+                type: :string,
+                default: nil,
+                full_name: N_('Choria Config File'),
+                description: N_(
+                  'Path on the smart proxy host to the Choria client configuration file. This file ' \
+                  'must be readable by the foreman-proxy user. When not set, the proxy uses a built-in default.'
+                )
+              setting 'openbolt_choria-mcollective-certname',
+                type: :string,
+                default: nil,
+                full_name: N_('Choria MCollective Certname'),
+                description: N_(
+                  'Override the MCollective certname for Choria client identity. When not set, the ' \
+                  'proxy derives this automatically from its SSL certificate.'
+                )
+              setting 'openbolt_choria-puppet-environment',
+                type: :string,
+                default: nil,
+                full_name: N_('Choria Puppet Environment'),
+                description: N_(
+                  'Puppet environment used by the Choria bolt_tasks agent to locate task files. ' \
+                  "Only applies when the Choria Task Agent is 'bolt_tasks'. Defaults to " \
+                  "'production' when not specified."
+                )
+              setting 'openbolt_choria-rpc-timeout',
+                type: :integer,
+                default: nil,
+                full_name: N_('Choria RPC Timeout'),
+                description: N_('Time in seconds to wait for RPC responses from target nodes.')
+              setting 'openbolt_choria-ssl-ca',
+                type: :string,
+                default: nil,
+                full_name: N_('Choria SSL CA'),
+                description: N_(
+                  'Path on the smart proxy host to the Choria CA certificate. This file must be ' \
+                  'readable by the foreman-proxy user.'
+                )
+              setting 'openbolt_choria-ssl-cert',
+                type: :string,
+                default: nil,
+                full_name: N_('Choria SSL Certificate'),
+                description: N_(
+                  'Path on the smart proxy host to the Choria client SSL certificate. This file ' \
+                  'must be readable by the foreman-proxy user.'
+                )
+              setting 'openbolt_choria-ssl-key',
+                type: :string,
+                default: nil,
+                full_name: N_('Choria SSL Key'),
+                description: N_(
+                  'Path on the smart proxy host to the Choria client SSL private key. This file ' \
+                  'must be readable by the foreman-proxy user.'
+                )
+              setting 'openbolt_choria-task-agent',
+                type: :string,
+                default: 'bolt_tasks',
+                full_name: N_('Choria Task Agent'),
+                description: N_(
+                  'Choria agent used to execute tasks on target nodes. Use the bolt_tasks agent for ' \
+                  'standard OpenBolt tasks, or the shell agent to run shell commands.'
+                ),
+                collection: proc { CHORIA_TASK_AGENTS }
+              setting 'openbolt_choria-task-timeout',
+                type: :integer,
+                default: nil,
+                full_name: N_('Choria Task Timeout'),
+                description: N_('Time in seconds to wait for task completion on target nodes.')
+
+              # SSH
               setting 'openbolt_host-key-check',
                 type: :boolean,
                 default: true,
@@ -95,7 +197,7 @@ module ForemanOpenbolt
                 description: N_('Whether to perform host key verification when connecting to targets over SSH')
               setting 'openbolt_private-key',
                 type: :string,
-                default: '',
+                default: nil,
                 full_name: N_('SSH Private Key'),
                 description: N_(
                   'Path on the smart proxy host to the private key used for SSH authentication. This key must be ' \
@@ -103,7 +205,7 @@ module ForemanOpenbolt
                 )
               setting 'openbolt_run-as',
                 type: :string,
-                default: '',
+                default: nil,
                 full_name: N_('SSH Run As User'),
                 description: N_(
                   'The user to run commands as on the target host. This requires that the user specified ' \
@@ -111,10 +213,12 @@ module ForemanOpenbolt
                 )
               setting 'openbolt_sudo-password',
                 type: :string,
-                default: '',
+                default: nil,
                 full_name: N_('SSH Sudo Password'),
                 description: N_('Password used for privilege escalation when using SSH'),
                 encrypted: true
+
+              # WinRM
               setting 'openbolt_ssl',
                 type: :boolean,
                 default: true,

data/lib/foreman_openbolt/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ForemanOpenbolt
-  VERSION = '1.1.0'
+  VERSION = '1.2.0'
 end

data/lib/proxy_api/openbolt.rb

@@ -52,15 +52,25 @@ module ProxyAPI
     end
 
     def parse_response(response, operation)
-      raise ProxyException.new(@url, nil, "No response from Smart Proxy during #{operation}") unless response
+      unless response
+        raise ProxyException.new(
+          @url, RuntimeError.new("No response from Smart Proxy during #{operation}"),
+          "No response from Smart Proxy during #{operation}"
+        )
+      end
 
       body = response.body
-      raise ProxyException.new(@url, nil, "Empty response body from Smart Proxy during #{operation}") if body.nil?
+      if body.nil?
+        raise ProxyException.new(
+          @url, RuntimeError.new("Empty response body from Smart Proxy during #{operation}"),
+          "Empty response body from Smart Proxy during #{operation}"
+        )
+      end
 
       JSON.parse(body)
     rescue JSON::ParserError => e
       raise ProxyException.new(
-        @url, nil,
+        @url, e,
         "Invalid JSON from Smart Proxy during #{operation}: #{e.message}. " \
         "Response body (first 500 chars): #{body.to_s[0..500]}"
       )

data/package.json

@@ -1,6 +1,6 @@
 {
   "name": "foreman_openbolt",
-  "version": "1.1.0",
+  "version": "1.2.0",
   "description": "OpenBolt integration into Foreman",
   "main": "index.js",
   "scripts": {