Skip to content

docs: CSRF settings precedence and audit of the csrf skills - #23

Merged
lmajano merged 2 commits into
mainfrom
claude/funny-mendel-eeim2l
Oct 9, 2026
Merged

lmajano merged 2 commits into
mainfrom
claude/funny-mendel-eeim2l

Conversation

@lmajano

@lmajano lmajano commented Oct 9, 2026

Copy link
Copy Markdown
Member

Summary

  • cbsecurity skill: new "CSRF Settings" section. Keys explicitly set in cbsecurity.csrf win over the cbcsrf module settings, other keys keep the cbcsrf value (matches fix: explicit cbcsrf module settings win over the cbsecurity csrf key coldbox-modules/cbsecurity#75).
  • cbcsrf and csrf-protection skills: audited against the cbcsrf module source and rewritten.
    • Settings now use the real keys: enableAutoVerifier, verifyExcludes, rotationTimeout, enableEndpoint, cacheStorage, enableAuthTokenRotator.
    • Helpers are the real mixins: csrfToken, csrfVerify, csrf, csrfField, csrfRotate, plus the @cbcsrf service.
    • Removed APIs that do not exist: CSRFService@cbcsrf, exemptFromCSRF(), generateCSRFToken(), verifyCSRFToken(), tokenKey, storageStrategy and others.
    • Documented how verification works: csrf field or x-csrf-token header, skipCsrf annotation, exceptions thrown, token endpoint.

Notes

Checked against cbcsrf 3.3.0 (ModuleConfig.cfc, helpers/Mixins.cfm, models/cbcsrf.cfc, interceptors/VerifyCsrf.cfc). Not run, docs only.

🤖 Generated with Claude Code

https://claude.ai/code/session_01E8yy4oK5vnaVucmYLg4UtY


Generated by Claude Code

claude added 2 commits October 9, 2026 18:11
… keys in the csrf skills

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01E8yy4oK5vnaVucmYLg4UtY
Replace helpers, service ids and annotations that do not exist with the real
cbcsrf API: csrfToken, csrfVerify, csrf, csrfField, csrfRotate, the @cbcsrf
service, the csrf field and x-csrf-token header, skipCsrf and verifyExcludes.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01E8yy4oK5vnaVucmYLg4UtY
@lmajano
lmajano merged commit 098b607 into main Oct 9, 2026
1 check 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