Skip to content

FOUR-32311: Prevent duplicate DevLink bundle operations#8921

Merged
eiresendez merged 1 commit into
epic/FOUR-32196from
bugfix/FOUR-32311
Jul 22, 2026
Merged

FOUR-32311: Prevent duplicate DevLink bundle operations#8921
eiresendez merged 1 commit into
epic/FOUR-32196from
bugfix/FOUR-32311

Conversation

@eiresendez

Copy link
Copy Markdown
Contributor

Issue & Reproduction Steps

DevLink allows an Update Bundle or Reinstall Bundle progress modal to be dismissed while the operation is still running. This makes the action available again and permits a duplicate job to be submitted, which the backend lock rejects with an "Already running!" error. To reproduce, navigate to Admin > DevLink > Local Bundles, start an Update Bundle or Reinstall Bundle operation, click outside the progress modal before completion, and attempt the same action again. No fixture is required.

Solution

  • Prevent the installation progress modal from being dismissed through the backdrop, Escape key, or header close button while an operation is active.
  • Guard against duplicate submissions and reset the progress component for every new installation attempt.
  • Display initial API request failures as terminal errors so the operation can be acknowledged and closed safely.
  • Release the operation lock in the interface only after the terminal result is closed.

How to Test

  1. Run npm run development.
  2. Navigate to Admin > DevLink > Local Bundles and start Update Bundle or Reinstall Bundle.
  3. Verify the progress modal has no header close button and cannot be dismissed by clicking the backdrop or pressing Escape.
  4. Verify only one operation is submitted while the modal is active.
  5. Allow the operation to complete and verify the Close button releases the interface.
  6. Start another legitimate operation and verify it begins with fresh progress state and completes without an "Already running!" error.

Related Tickets & Packages

@eiresendez eiresendez self-assigned this Jul 21, 2026
@processmaker-sonarqube

Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@eiresendez
eiresendez requested a review from CarliPinell July 21, 2026 20:20

@CarliPinell CarliPinell left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved.
No observations were found

@CarliPinell CarliPinell left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved.
No observations were found

@eiresendez
eiresendez merged commit e801bba into epic/FOUR-32196 Jul 22, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants