diff options
Diffstat (limited to 'internal')
| -rw-r--r-- | internal/mobius/api.go | 3 | ||||
| -rw-r--r-- | internal/mobius/threaded_news.go | 2 | ||||
| -rw-r--r-- | internal/mobius/transaction_handlers.go | 5 |
3 files changed, 7 insertions, 3 deletions
diff --git a/internal/mobius/api.go b/internal/mobius/api.go index 4dfc575..0a20d01 100644 --- a/internal/mobius/api.go +++ b/internal/mobius/api.go @@ -263,7 +263,8 @@ func (srv *APIServer) ReloadHandler(reloadFunc func()) func(w http.ResponseWrite func (srv *APIServer) RenderStats(w http.ResponseWriter, _ *http.Request) { u, err := json.Marshal(srv.hlServer.CurrentStats()) if err != nil { - panic(err) + http.Error(w, "failed to marshal stats", http.StatusInternalServerError) + return } _, _ = w.Write(u) diff --git a/internal/mobius/threaded_news.go b/internal/mobius/threaded_news.go index e9d9c22..67e8282 100644 --- a/internal/mobius/threaded_news.go +++ b/internal/mobius/threaded_news.go @@ -195,7 +195,7 @@ func (n *ThreadedNewsYAML) DeleteArticle(newsPath []string, articleID uint32, _ return n.writeFile() } -func (n *ThreadedNewsYAML) ListArticles(newsPath []string) hotline.NewsArtListData { +func (n *ThreadedNewsYAML) ListArticles(newsPath []string) (hotline.NewsArtListData, error) { n.mu.Lock() defer n.mu.Unlock() diff --git a/internal/mobius/transaction_handlers.go b/internal/mobius/transaction_handlers.go index d8c4cb5..5181e15 100644 --- a/internal/mobius/transaction_handlers.go +++ b/internal/mobius/transaction_handlers.go @@ -1319,7 +1319,10 @@ func HandleGetNewsArtNameList(cc *hotline.ClientConn, t *hotline.Transaction) (r return res } - nald := cc.Server.ThreadedNewsMgr.ListArticles(pathStrs) + nald, err := cc.Server.ThreadedNewsMgr.ListArticles(pathStrs) + if err != nil { + return res + } b, err := io.ReadAll(&nald) if err != nil { |