Skip to content

feat(auth): add middleware and route decorator to auth installer#2187

Merged
brendt merged 4 commits into
3.xfrom
auth-middleware
Jun 25, 2026
Merged

feat(auth): add middleware and route decorator to auth installer#2187
brendt merged 4 commits into
3.xfrom
auth-middleware

Conversation

@brendt

@brendt brendt commented Jun 24, 2026

Copy link
Copy Markdown
Member

No description provided.

@brendt brendt requested a review from innocenzi as a code owner June 24, 2026 08:26
@brendt

brendt commented Jun 24, 2026

Copy link
Copy Markdown
Member Author

As with everything auth, it's not possible to directly ship these with the framework because likely the user wants to make changes to the very simple basic implementation. Rather, it's a starting point for developers to build on

@brendt

brendt commented Jun 24, 2026

Copy link
Copy Markdown
Member Author

@innocenzi wdyt?

@innocenzi

Copy link
Copy Markdown
Member

Besides the names, very good idea!

I suggestAuthMiddleware -> MustBeAuthenticatedMiddleware and #[Auth] -> #[MustBeAuthenticated]

I have an issue with Auth as a name, it's too generic and doesn't convey what it does

@brendt brendt merged commit 527f0ae into 3.x Jun 25, 2026
75 checks passed
@brendt brendt deleted the auth-middleware branch June 25, 2026 09:06
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