get rid of some unnecessary panics

api/router.go

@@ -2,16 +2,12 @@ package api
 
 import (
 	"encoding/json"
-	"errors"
 	"fmt"
 	"net/http"
 
-	"github.com/getsentry/sentry-go"
 	"github.com/julienschmidt/httprouter"
-	"github.com/sirupsen/logrus"
 	"github.com/t2bot/matrix-media-repo/api/_responses"
 	"github.com/t2bot/matrix-media-repo/api/_routers"
-	"github.com/t2bot/matrix-media-repo/util"
 )
 
 func buildPrimaryRouter() *httprouter.Router {

api/webserver.go

@@ -80,7 +80,7 @@ func Stop() {
 		ctx, cancel := context.WithTimeout(context.Background(), 5*time.Second)
 		defer cancel()
 		if err := srv.Shutdown(ctx); err != nil {
-			panic(err)
+			logrus.Fatalf("Could not gracefully shutdown the server: %v", err)
 		}
 	}
 }

common/assets/process.go

@@ -24,7 +24,7 @@ func SetupMigrations(givenMigrationsPath string) {
 	if !exists {
 		tempMigrations, err = os.MkdirTemp(os.TempDir(), "media-repo-migrations")
 		if err != nil {
-			panic(err)
+			logrus.Fatalf("Failed to create temporary migrations directory: %v", err)
 		}
 		logrus.Info("Migrations path doesn't exist - attempting to unpack from compiled data")
 		extractPrefixTo("migrations", tempMigrations)