Pass request to withMiddlewareAuth's isPermitted function

[413n]

Feature request

Is your feature request related to a problem? Please describe.

I like the functionality of the withMiddlewareAuth helper, but in my use case I'm a bit limited. I would like to have multiple paths where I check the user authentication and for each I would like to check for a different user role.

Describe the solution you'd like

Referring to the following code, I would suggest to pass the request in the isPermitted function, so that we could check also for the url and be more specific.

https://github.com/supabase/auth-helpers/blob/main/examples/nextjs/middleware.ts#L1-L15

[NanoCode012]

Yes! I have the same use case (check if user has role to access certain page). I used to do it via getServerSideProp, but it is inefficient and doesn't allow page caching.

With the new 12.2 nextjs middleware, we could use a single middleware to check, so it's faster.

The lines to change should be:

auth-helpers/packages/nextjs/src/middleware/withMiddlewareAuth.ts

Line 35 in d65b8b4

isPermitted: (user: User) => Promise<boolean>;

and

auth-helpers/packages/nextjs/src/middleware/withMiddlewareAuth.ts

Line 140 in d65b8b4

!(await options.authGuard.isPermitted(authResult.user))

I may open a PR targeting the 0.2.9 tag as I'm still using Supabase v1. However, I'm not sure which branch I should target to in my PR.

[NanoCode012]

Sorry for my late follow up, I have written the code for 0.2.9, but it should also be applicable to latest main branch also with some modification.

NanoCode012@93405a2

Changes from 0.2.9: @supabase/[email protected]:supabase-auth-helpers:feat/add-middleware-req

If someone could point me to how I could also have this update for Supabase v1, that'd be great. If not, please tell me so I could adapt the code for latest main branch.

[NanoCode012]

I just saw a new PR #364 which would solve this issue. Looking forward to it!