Skip to content

Add AutoClean exclusion controls#12

Merged
lmajano merged 1 commit into
developmentfrom
codex/cbantisamy-optional-autoclean-interceptor
Jul 16, 2026
Merged

Add AutoClean exclusion controls#12
lmajano merged 1 commit into
developmentfrom
codex/cbantisamy-optional-autoclean-interceptor

Conversation

@elpete

@elpete elpete commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds AutoClean exclusion controls so consumers can skip cleaning selected request collection keys without replacing the interceptor.

  • Adds autoCleanExclusions module settings for global, event, handler wildcard, module handler wildcard, and module-wide wildcard exclusions.
  • Adds action-level antisamyAutoCleanExclusions metadata support for handler methods.
  • Merges configured, annotated, and existing request-private exclusions before cleaning.
  • Documents both configuration and annotation usage.
  • Adds TestBox coverage, including a hyphenated module event fixture for api-v1:Trips.update.

Validation

  • box run-script format:check
  • box testbox run

@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown

lucee@5 ColdBox be Test Results

11 tests  +3   11 ✅ +3   1s ⏱️ -1s
 2 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 784ae29. ± Comparison against base commit 351a766.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown

lucee@5 ColdBox ^7.0.0 Test Results

11 tests  +3   11 ✅ +3   1s ⏱️ -1s
 2 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 784ae29. ± Comparison against base commit 351a766.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown

lucee@6 ColdBox be Test Results

11 tests  +3   11 ✅ +3   1s ⏱️ -1s
 2 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 784ae29. ± Comparison against base commit 351a766.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown

boxlang-cfml@1 ColdBox be Test Results

11 tests  +3   11 ✅ +3   2s ⏱️ -1s
 2 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 784ae29. ± Comparison against base commit 351a766.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown

boxlang-cfml@1 ColdBox ^7.0.0 Test Results

11 tests  +3   11 ✅ +3   2s ⏱️ -1s
 2 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 784ae29. ± Comparison against base commit 351a766.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown

adobe@2023 ColdBox be Test Results

11 tests  +3   11 ✅ +3   1s ⏱️ -1s
 2 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 784ae29. ± Comparison against base commit 351a766.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown

adobe@2023 ColdBox ^7.0.0 Test Results

11 tests  +3   11 ✅ +3   0s ⏱️ ±0s
 2 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 784ae29. ± Comparison against base commit 351a766.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown

adobe@2021 ColdBox be Test Results

11 tests  +3   11 ✅ +3   1s ⏱️ -1s
 2 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 784ae29. ± Comparison against base commit 351a766.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown

adobe@2021 ColdBox ^7.0.0 Test Results

11 tests  +3   11 ✅ +3   0s ⏱️ ±0s
 2 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 784ae29. ± Comparison against base commit 351a766.

♻️ This comment has been updated with latest results.

@elpete
elpete marked this pull request as ready for review July 16, 2026 03:08
@lmajano
lmajano merged commit a277087 into development Jul 16, 2026
30 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