From e30cc95b08342be3237e3f21ce628afe8b09387f Mon Sep 17 00:00:00 2001
From: Asaf Gartner <asaf@handmadedev.org>
Date: Fri, 23 Jul 2021 06:22:31 +0300
Subject: [PATCH] Added request rejection page

---
 src/templates/src/reject.html | 17 +++++++++++++++++
 src/website/podcast.go        | 25 +++++++++++++------------
 src/website/routes.go         | 17 +++++++++++++++++
 3 files changed, 47 insertions(+), 12 deletions(-)
 create mode 100644 src/templates/src/reject.html

diff --git a/src/templates/src/reject.html b/src/templates/src/reject.html
new file mode 100644
index 0000000..4684762
--- /dev/null
+++ b/src/templates/src/reject.html
@@ -0,0 +1,17 @@
+{{ template "base.html" . }}
+
+{{ define "content" }}
+<div class="description">
+	<p>
+		<span class="big">Hi there, {{ if .User }}{{ .User.Name }}{{ else }}visitor{{ end }}!</span>
+	</p>
+	<p>
+		<span class="big">Your request was rejected for the following reason:</span><br />
+		<p class="b">{{ .RejectReason }}</p>
+		<p>
+			<a href="javascript:history.back();">Go back</a>
+		</p>
+	</p>
+</div>
+{{ end }}
+
diff --git a/src/website/podcast.go b/src/website/podcast.go
index 32d37f7..5a41edd 100644
--- a/src/website/podcast.go
+++ b/src/website/podcast.go
@@ -142,13 +142,11 @@ func PodcastEditSubmit(c *RequestContext) ResponseData {
 
 	title := c.Req.Form.Get("title")
 	if len(strings.TrimSpace(title)) == 0 {
-		// TODO(asaf): Report this back to the user
-		return ErrorResponse(http.StatusInternalServerError, oops.New(nil, "Missing title"))
+		return RejectRequest(c, "Podcast title is empty")
 	}
 	description := c.Req.Form.Get("description")
 	if len(strings.TrimSpace(description)) == 0 {
-		// TODO(asaf): Report this back to the user
-		return ErrorResponse(http.StatusInternalServerError, oops.New(nil, "Missing description"))
+		return RejectRequest(c, "Podcast description is empty")
 	}
 	podcastImage, header, err := c.Req.FormFile("podcast_image")
 	imageFilename := ""
@@ -159,21 +157,18 @@ func PodcastEditSubmit(c *RequestContext) ResponseData {
 	}
 	if header != nil {
 		if header.Size > maxFileSize {
-			// TODO(asaf): Report this back to the user
-			return ErrorResponse(http.StatusInternalServerError, oops.New(nil, "Filesize too big"))
+			return RejectRequest(c, fmt.Sprintf("Image filesize too big. Max size: %d bytes", maxFileSize))
 		} else {
 			c.Perf.StartBlock("PODCAST", "Decoding image")
 			config, format, err := image.DecodeConfig(podcastImage)
 			c.Perf.EndBlock()
 			if err != nil {
-				// TODO(asaf): Report this back to the user
-				return ErrorResponse(http.StatusInternalServerError, oops.New(err, "Can't parse podcast logo"))
+				return RejectRequest(c, "Image type not supported")
 			}
 			imageWidth = config.Width
 			imageHeight = config.Height
 			if imageWidth == 0 || imageHeight == 0 {
-				// TODO(asaf): Report this back to the user
-				return ErrorResponse(http.StatusInternalServerError, oops.New(err, "Invalid image size"))
+				return RejectRequest(c, "Image has zero size")
 			}
 
 			imageFilename = fmt.Sprintf("podcast/%s/logo%d.%s", c.CurrentProject.Slug, time.Now().UTC().Unix(), format)
@@ -435,11 +430,17 @@ func PodcastEpisodeSubmit(c *RequestContext) ResponseData {
 
 	c.Req.ParseForm()
 	title := c.Req.Form.Get("title")
+	if len(strings.TrimSpace(title)) == 0 {
+		return RejectRequest(c, "Episode title is empty")
+	}
 	description := c.Req.Form.Get("description")
+	if len(strings.TrimSpace(description)) == 0 {
+		return RejectRequest(c, "Episode description is empty")
+	}
 	episodeNumberStr := c.Req.Form.Get("episode_number")
 	episodeNumber, err := strconv.Atoi(episodeNumberStr)
 	if err != nil {
-		return ErrorResponse(http.StatusInternalServerError, oops.New(err, "Failed to parse episode number"))
+		return RejectRequest(c, "Episode number can't be parsed")
 	}
 	episodeFile := c.Req.Form.Get("episode_file")
 	found = false
@@ -451,7 +452,7 @@ func PodcastEpisodeSubmit(c *RequestContext) ResponseData {
 	}
 
 	if !found {
-		return ErrorResponse(http.StatusInternalServerError, oops.New(err, "User-provided episode filename doesn't match existing files"))
+		return RejectRequest(c, "Requested episode file not found")
 	}
 
 	c.Perf.StartBlock("MP3", "Parsing mp3 file for duration")
diff --git a/src/website/routes.go b/src/website/routes.go
index 07f8669..d37a0b3 100644
--- a/src/website/routes.go
+++ b/src/website/routes.go
@@ -318,6 +318,23 @@ func FourOhFour(c *RequestContext) ResponseData {
 	return res
 }
 
+type RejectData struct {
+	templates.BaseData
+	RejectReason string
+}
+
+func RejectRequest(c *RequestContext, reason string) ResponseData {
+	var res ResponseData
+	err := res.WriteTemplate("reject.html", RejectData{
+		BaseData:     getBaseData(c),
+		RejectReason: reason,
+	}, c.Perf)
+	if err != nil {
+		return ErrorResponse(http.StatusInternalServerError, oops.New(err, "Failed to render reject template"))
+	}
+	return res
+}
+
 func LoadCommonWebsiteData(c *RequestContext) (bool, ResponseData) {
 	c.Perf.StartBlock("MIDDLEWARE", "Load common website data")
 	defer c.Perf.EndBlock()