mstar/linstrom
1
0
Fork 0
Code Issues 3 Pull requests Projects Releases Packages Wiki Activity Actions

Fix: Better error handling

Browse source 
Session insertion middleware now has better error handling for fetching
an account based on a session. Not found is treated as not an error
since it is likely to happen that a session is used post account
deletion
This commit is contained in:
Melody Becker 2024-11-07 10:42:41 +01:00
parent ad68e97eaf
commit 9169cec4bb
1 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
server/middlewares.go
View file

@ -132,7 +132,13 @@ func checkSessionMiddleware(handler http.Handler) http.Handler {
return return
} }
acc, err := store.FindAccountByPasskeyId(session.UserID) acc, err := store.FindAccountByPasskeyId(session.UserID)
if err != nil { switch err {
case storage.ErrEntryNotFound:
log.Info().Msg("No account for the given passkey id found. It probably got deleted")
handler.ServeHTTP(w, r)
return
case nil:
default:
// Failed to get account for passkey id. Log, then move on as if no session is set // Failed to get account for passkey id. Log, then move on as if no session is set
log.Error(). log.Error().
Err(err). Err(err).