Skip to content

Commit 6df22db

Browse files
fix: Convert some more panics to errors messages
1 parent ccc6245 commit 6df22db

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

internal/session/handle.go

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ package session
22

33
import (
44
"context"
5+
"fmt"
56
"runtime/pprof"
67
"strconv"
78

@@ -87,7 +88,7 @@ func (s *Session) handleCommand(
8788
return s.handleSelectedCommand(ctx, tag, cmd, ch, profiler)
8889

8990
default:
90-
panic("bad command")
91+
return fmt.Errorf("bad command")
9192
}
9293
}
9394

@@ -121,7 +122,7 @@ func (s *Session) handleAnyCommand(
121122
return s.handleIDGet(ctx, tag, ch)
122123

123124
default:
124-
panic("bad command")
125+
return fmt.Errorf("bad command")
125126
}
126127
}
127128

@@ -144,7 +145,7 @@ func (s *Session) handleNotAuthenticatedCommand(
144145
return s.handleLogin(ctx, tag, cmd.GetLogin(), ch)
145146

146147
default:
147-
panic("bad command")
148+
return fmt.Errorf("bad command")
148149
}
149150
}
150151

@@ -230,7 +231,7 @@ func (s *Session) handleAuthenticatedCommand(
230231
return s.handleAppend(ctx, tag, cmd.GetAppend(), ch)
231232

232233
default:
233-
panic("bad command")
234+
return fmt.Errorf("bad command")
234235
}
235236
}
236237

@@ -336,6 +337,6 @@ func (s *Session) handleWithMailbox(
336337
return s.handleMove(ctx, tag, cmd.GetMove(), mailbox, ch)
337338

338339
default:
339-
panic("bad command")
340+
return fmt.Errorf("bad command")
340341
}
341342
}

0 commit comments

Comments
 (0)