Skip to content

FOUR-32498 [Octane] MEDIUM — Accumulating State "addons" - #8961

Merged
pmPaulis merged 1 commit into
feature/FOUR-32464from
feature/FOUR-32498
Aug 3, 2026
Merged

FOUR-32498 [Octane] MEDIUM — Accumulating State "addons"#8961
pmPaulis merged 1 commit into
feature/FOUR-32464from
feature/FOUR-32498

Conversation

@gproly

@gproly gproly commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Description:
Fix Octane state leak by moving controller addons from static trait property to singleton registry

Replace HasControllerAddons static $addons with ControllerAddonsRegistry to prevent addon accumulation across Octane requests while keeping the existing registerAddon() and getPluginAddons() API unchanged.

Related tickets:
https://processmaker.atlassian.net/browse/FOUR-32498

Description:
Fix Octane state leak by moving controller addons from static trait property to singleton registry

Replace `HasControllerAddons` static `$addons` with `ControllerAddonsRegistry` to prevent addon accumulation across Octane requests while keeping the existing `registerAddon()` and `getPluginAddons()` API unchanged.

Related tickets:
https://processmaker.atlassian.net/browse/FOUR-32498
@gproly
gproly requested a review from devmiguelangel July 31, 2026 19:44
@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
0.0% Duplication on New Code

See analysis details on SonarQube

@pmPaulis
pmPaulis changed the base branch from feature/FOUR-30918 to feature/FOUR-32464 July 31, 2026 20:11
@pmPaulis
pmPaulis merged commit 78e5b1b into feature/FOUR-32464 Aug 3, 2026
14 of 15 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.

3 participants