Skip to content

Skills: route middleware, registerMiddleware(), group meta and middleware testing - #20

Merged
lmajano merged 1 commit into
mainfrom
claude/funny-mendel-eeim2l
Oct 8, 2026
Merged

lmajano merged 1 commit into
mainfrom
claude/funny-mendel-eeim2l

Conversation

@lmajano

@lmajano lmajano commented Oct 8, 2026

Copy link
Copy Markdown
Member

Updates three skills for the route middleware work across ColdBox and cbsecurity.

Changes

Version labels follow the existing "ColdBox 8.2.0+" convention. Merge after the linked ColdBox and cbsecurity PRs ship, or keep the 8.3 and 3.9 labels as they are.

Not changed, but wrong

While in modules/cbsecurity/SKILL.md I saw other statements that do not match the cbsecurity source. I left them out of this PR to keep it to the middleware work:

  • SecurityService@cbsecurity and getCurrentUser() (the model is CBSecurity@cbsecurity and the method is getUser())
  • hasRole(), hasPermission() on the security object (it has has(), all(), none(), secure() and friends)
  • firewall keys autoSecureAll and authenticate, and a boolean whiteList (the rule keys are secureList, whitelist, permissions, roles, action, and so on)
  • the IUserService method list and the cfauth provider

I can send a follow-up PR that corrects those against the source if wanted.

🤖 Generated with Claude Code

https://claude.ai/code/session_01E8yy4oK5vnaVucmYLg4UtY


Generated by Claude Code

- routing-development: registerMiddleware() (ColdBox 8.3+), group() meta
  inheritance (8.3+), and fix the group example that used a middleware name
  that does not exist
- cbsecurity: Route Middleware section (Authenticated, Authorized, JwtAuth,
  BasicAuth, route meta) and replace the Securing API Routes example, which
  used a non-existent withMiddleware( "JwtAuthFilter@cbsecurity" )
- testing-integration: execute() and get()/post() run route middleware (8.3+)

Co-Authored-By: Claude <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01E8yy4oK5vnaVucmYLg4UtY
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