root: fix linting errors

Signed-off-by: Jens Langhammer <jens.langhammer@beryju.org>

cmd/server/main.go

@@ -2,8 +2,6 @@ package main
 
 import (
 	"fmt"
-	"net/url"
-	"time"
 
 	"github.com/getsentry/sentry-go"
 	log "github.com/sirupsen/logrus"
@@ -11,8 +9,6 @@ import (
 	"goauthentik.io/internal/config"
 	"goauthentik.io/internal/constants"
 	"goauthentik.io/internal/gounicorn"
-	"goauthentik.io/internal/outpost/ak"
-	"goauthentik.io/internal/outpost/proxy"
 	"goauthentik.io/internal/web"
 )
 
@@ -21,18 +17,27 @@ var running = true
 func main() {
 	log.SetLevel(log.DebugLevel)
 	config.DefaultConfig()
-	config.LoadConfig("./authentik/lib/default.yml")
-	config.LoadConfig("./local.env.yml")
+	err := config.LoadConfig("./authentik/lib/default.yml")
+	if err != nil {
+		log.WithError(err).Warning("failed to load default config")
+	}
+	err = config.LoadConfig("./local.env.yml")
+	if err != nil {
+		log.WithError(err).Debug("failed to local config")
+	}
 	config.ConfigureLogger()
 
 	if config.G.ErrorReporting.Enabled {
-		sentry.Init(sentry.ClientOptions{
+		err := sentry.Init(sentry.ClientOptions{
 			Dsn:              "https://a579bb09306d4f8b8d8847c052d3a1d3@sentry.beryju.org/8",
 			AttachStacktrace: true,
 			TracesSampleRate: 0.6,
 			Release:          fmt.Sprintf("authentik@%s", constants.VERSION),
 			Environment:      config.G.ErrorReporting.Environment,
 		})
+		if err != nil {
+			log.WithError(err).Warning("failed to init sentry")
+		}
 	}
 
 	ex := common.Init()
@@ -58,45 +63,45 @@ func main() {
 	}
 }
 
-func attemptStartBackend(g *gounicorn.GoUnicorn) error {
+func attemptStartBackend(g *gounicorn.GoUnicorn) {
 	for {
 		err := g.Start()
 		if !running {
-			return nil
+			return
 		}
 		log.WithField("logger", "authentik.g").WithError(err).Warning("gunicorn process died, restarting")
 	}
 }
 
-func attemptProxyStart(u *url.URL) error {
-	maxTries := 100
-	attempt := 0
-	for {
-		log.WithField("logger", "authentik").Debug("attempting to init outpost")
-		ac := ak.NewAPIController(*u, config.G.SecretKey)
-		if ac == nil {
-			attempt += 1
-			time.Sleep(1 * time.Second)
-			if attempt > maxTries {
-				break
-			}
-			continue
-		}
-		ac.Server = proxy.NewServer(ac)
-		err := ac.Start()
-		log.WithField("logger", "authentik").Debug("attempting to start outpost")
-		if err != nil {
-			attempt += 1
-			time.Sleep(5 * time.Second)
-			if attempt > maxTries {
-				break
-			}
-			continue
-		}
-		if !running {
-			ac.Shutdown()
-			return nil
-		}
-	}
-	return nil
-}
+// func attemptProxyStart(u *url.URL) error {
+// 	maxTries := 100
+// 	attempt := 0
+// 	for {
+// 		log.WithField("logger", "authentik").Debug("attempting to init outpost")
+// 		ac := ak.NewAPIController(*u, config.G.SecretKey)
+// 		if ac == nil {
+// 			attempt += 1
+// 			time.Sleep(1 * time.Second)
+// 			if attempt > maxTries {
+// 				break
+// 			}
+// 			continue
+// 		}
+// 		ac.Server = proxy.NewServer(ac)
+// 		err := ac.Start()
+// 		log.WithField("logger", "authentik").Debug("attempting to start outpost")
+// 		if err != nil {
+// 			attempt += 1
+// 			time.Sleep(5 * time.Second)
+// 			if attempt > maxTries {
+// 				break
+// 			}
+// 			continue
+// 		}
+// 		if !running {
+// 			ac.Shutdown()
+// 			return nil
+// 		}
+// 	}
+// 	return nil
+// }

internal/gounicorn/gounicorn.go

@@ -58,7 +58,10 @@ func (g *GoUnicorn) Start() error {
 	return g.p.Run()
 }
 
-func (g *GoUnicorn) Kill() error {
+func (g *GoUnicorn) Kill() {
 	g.killed = true
-	return g.p.Process.Kill()
+	err := g.p.Process.Kill()
+	if err != nil {
+		g.log.WithError(err).Warning("failed to kill gunicorn")
+	}
 }

internal/web/middleware_sentry.go

@@ -5,10 +5,12 @@ import (
 	"net/http"
 
 	sentryhttp "github.com/getsentry/sentry-go/http"
+	log "github.com/sirupsen/logrus"
 )
 
 func recoveryMiddleware() func(next http.Handler) http.Handler {
 	sentryHandler := sentryhttp.New(sentryhttp.Options{})
+	l := log.WithField("logger", "authentik.sentry")
 	return func(next http.Handler) http.Handler {
 		sentryHandler.Handle(next)
 		return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
@@ -20,6 +22,7 @@ func recoveryMiddleware() func(next http.Handler) http.Handler {
 				}
 				err := re.(error)
 				if err != nil {
+					l.WithError(err).Warning("global panic handler")
 					jsonBody, _ := json.Marshal(struct {
 						Successful bool
 						Error      string
@@ -30,7 +33,10 @@ func recoveryMiddleware() func(next http.Handler) http.Handler {
 
 					w.Header().Set("Content-Type", "application/json")
 					w.WriteHeader(http.StatusInternalServerError)
-					w.Write(jsonBody)
+					_, err := w.Write(jsonBody)
+					if err != nil {
+						l.WithError(err).Warning("Failed to write sentry error body")
+					}
 				}
 			}()
 		})

internal/web/web.go

@@ -66,7 +66,10 @@ func (ws *WebServer) listenPlain() {
 	ws.serve(ln)
 
 	ws.log.WithField("addr", config.G.Web.Listen).Info("Running")
-	http.ListenAndServe(config.G.Web.Listen, ws.m)
+	err = http.ListenAndServe(config.G.Web.Listen, ws.m)
+	if err != nil && !errors.Is(err, http.ErrServerClosed) {
+		ws.log.Errorf("ERROR: http.Serve() - %s", err)
+	}
 }
 
 func (ws *WebServer) serve(listener net.Listener) {

internal/web/web_ssl.go

@@ -23,6 +23,7 @@ func (ws *WebServer) listenTLS() {
 	ln, err := net.Listen("tcp", config.G.Web.ListenTLS)
 	if err != nil {
 		ws.log.WithError(err).Fatalf("failed to listen")
+		return
 	}
 	ws.log.WithField("addr", config.G.Web.ListenTLS).Info("Running")
 

internal/web/web_static.go

@@ -29,12 +29,18 @@ func (ws *WebServer) configureStatic() {
 	ws.lh.Path("/robots.txt").HandlerFunc(func(rw http.ResponseWriter, r *http.Request) {
 		rw.Header()["Content-Type"] = []string{"text/plain"}
 		rw.WriteHeader(200)
-		rw.Write(staticWeb.RobotsTxt)
+		_, err := rw.Write(staticWeb.RobotsTxt)
+		if err != nil {
+			ws.log.WithError(err).Warning("failed to write response")
+		}
 	})
 	ws.lh.Path("/.well-known/security.txt").HandlerFunc(func(rw http.ResponseWriter, r *http.Request) {
 		rw.Header()["Content-Type"] = []string{"text/plain"}
 		rw.WriteHeader(200)
-		rw.Write(staticWeb.SecurityTxt)
+		_, err := rw.Write(staticWeb.SecurityTxt)
+		if err != nil {
+			ws.log.WithError(err).Warning("failed to write response")
+		}
 	})
 }