Added error case for when DownloadEpisode is called via API and podcast is already downloaded

podcasts/podcasts.go

@@ -358,6 +358,10 @@ func (p *Podcasts) DownloadEpisode(episodeID int) error {
 		log.Printf("Already downloading podcast episode with id %d", episodeID)
 		return nil
 	}
+	if podcastEpisode.Status == db.PodcastEpisodeStatusCompleted {
+		log.Printf("Already downloaded podcast episode with id %d", episodeID)
+		return nil
+	}
 	podcastEpisode.Status = db.PodcastEpisodeStatusDownloading
 	p.db.Save(&podcastEpisode)
 	// nolint: bodyclose

server/ctrlbase/ctrl.go

@@ -56,8 +56,14 @@ func (c *Controller) Path(rel string) string {
 }
 
 func (c *Controller) BaseURL(r *http.Request) string {
+	var protocol string
+	if r.TLS == nil {
+		protocol = "http"
+	} else {
+		protocol = "https"
+	}
 	scheme := firstExisting(
-		"http", // fallback
+		protocol, // fallback
 		r.Header.Get("X-Forwarded-Proto"),
 		r.Header.Get("X-Forwarded-Scheme"),
 		r.URL.Scheme,

server/ctrlsubsonic/handlers_common.go

@@ -54,7 +54,7 @@ func (c *Controller) ServeScrobble(r *http.Request) *spec.Response {
 
 	optStamp := params.GetOrTime("time", time.Now())
 	optSubmission := params.GetOrBool("submission", true)
-
+	
 	if err := streamUpdateStats(c.DB, user.ID, track.Album.ID, optStamp); err != nil {
 		return spec.NewError(0, "error updating stats: %v", err)
 	}