From 9f634ce015a9c984655a09a3c33682c8ba138dfd Mon Sep 17 00:00:00 2001
From: Drew DeVault <sir@cmpwn.com>
Date: Wed, 22 Sep 2021 09:04:18 +0200
Subject: [PATCH] server/email: handle errors out of GQL context

---
 server/email.go | 33 ++++++++++++++++++++++-----------
 1 file changed, 22 insertions(+), 11 deletions(-)

diff --git a/server/email.go b/server/email.go
index c7c245f77af7d5c1c6a9e8de3568eeb109b7708f..1df5cf553f52d512b9b1d7fc09159e40878078a3 100644
--- a/server/email.go
+++ b/server/email.go
@@ -2,13 +2,14 @@ package server
 
 import (
 	"context"
-	"errors"
 	"encoding/json"
+	"errors"
 	"fmt"
+	"io"
 	"log"
-	gomail "net/mail"
 	"runtime"
 	"strings"
+	gomail "net/mail"
 
 	"github.com/99designs/gqlgen/graphql"
 	"github.com/emersion/go-message/mail"
@@ -67,14 +68,23 @@ func EmailRecover(ctx context.Context, _origErr interface{}) error {
 	addr := mail.Address(*rcpt)
 	header.SetAddressList("To", []*mail.Address{&addr})
 
-	quser := auth.ForContext(ctx)
-	octx := graphql.GetOperationContext(ctx)
-	vars, err := json.Marshal(octx.Variables)
-	if err != nil {
-		vars = []byte{}[:]
-	}
-	reader := strings.NewReader(
-		fmt.Sprintf(`Error occured processing GraphQL request:
+	var reader io.Reader
+	func() {
+		defer func() {
+			if err := recover(); err != nil {
+				reader = strings.NewReader(fmt.Sprintf(`An error occured outside of the GraphQL context:
+
+				%s`, string(stack[:i])))
+			}
+		}()
+		quser := auth.ForContext(ctx)
+		octx := graphql.GetOperationContext(ctx)
+		vars, err := json.Marshal(octx.Variables)
+		if err != nil {
+			vars = []byte{}[:]
+		}
+		reader = strings.NewReader(
+			fmt.Sprintf(`Error occured processing GraphQL request:
 
 %v
 
@@ -89,7 +99,8 @@ With these variables:
 The following stack trace was produced:
 
 %s`, origErr, quser.Username, quser.Email, octx.RawQuery,
-	string(vars), string(stack[:i])))
+		string(vars), string(stack[:i])))
+	}()
 
 	email.EnqueueStd(ctx, header, reader, nil)
 	return fmt.Errorf("internal system error")