Skip to content

Commit

Permalink
fix: fetch hook is called even is user is not set (#209)
Browse files Browse the repository at this point in the history
* fix: fetch hook is called even is user is not set

* up
  • Loading branch information
atinux authored Oct 7, 2024
1 parent c95cb73 commit 457b6b1
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
3 changes: 1 addition & 2 deletions src/runtime/server/api/session.get.ts
Original file line number Diff line number Diff line change
@@ -1,13 +1,12 @@
import { eventHandler } from 'h3'
import { getUserSession, sessionHooks } from '../utils/session'
import type { UserSessionRequired } from '#auth-utils'

export default eventHandler(async (event) => {
const session = await getUserSession(event)

// If session is not empty, call fetch hook
if (Object.keys(session).length > 0) {
await sessionHooks.callHookParallel('fetch', session as UserSessionRequired, event)
await sessionHooks.callHookParallel('fetch', session, event)
}

const { secure, ...data } = session
Expand Down
2 changes: 1 addition & 1 deletion src/runtime/server/utils/session.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ export interface SessionHooks {
* - Add extra properties to the session
* - Throw an error if the session could not be verified (with a database for example)
*/
fetch: (session: UserSessionRequired, event: H3Event) => void | Promise<void>
fetch: (session: UserSession, event: H3Event) => void | Promise<void>
/**
* Called before clearing the session
*/
Expand Down

0 comments on commit 457b6b1

Please sign in to comment.