Skip to content

Commit

Permalink
fix(vite): Rename serverAuthContext to serverAuthState (#10653)
Browse files Browse the repository at this point in the history
**Problem**
#10643 did some renaming but
looks like there was just one more `serverAuthContext` that needed
switched over to `serverAuthState`. I noticed this just because the CI
was failing on renovate PRs.

**Note**
I'll merge this now and we can discuss retroactively if I misinterpreted
anything since I haven't been in the middleware space as much. It's
awesome work can't wait for all this to get out into stable for people
to enjoy!
  • Loading branch information
Josh-Walker-GM authored May 20, 2024
1 parent ab3fb1a commit 5b50203
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion packages/vite/src/middleware/invokeMiddleware.ts
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ export const invoke = async (
// @TODO catch the error here, and see if its a short-circuit
// A shortcircuit will prevent execution of all other middleware down the chain, and prevent react rendering
if (e instanceof MiddlewareShortCircuit) {
return [e.mwResponse, mwReq.serverAuthContext.get()]
return [e.mwResponse, mwReq.serverAuthState.get()]
}

console.error('Error executing middleware > \n')
Expand Down

0 comments on commit 5b50203

Please sign in to comment.