fix YT bug (blocking channel)

Author: Tanq16

downloaders/youtube/youtube.go

@@ -140,7 +140,7 @@ func removeExtension(filePath string) string {
 	return filePath[:len(filePath)-len(ext)]
 }
 
-func DownloadYouTubeVideo(url, outputPathPre, format, dType string, progressCh chan<- int64, outputCh chan<- []string) error {
+func DownloadYouTubeVideo(url, outputPathPre, format, dType string, outputCh chan<- []string) error {
 	outputPath := removeExtension(outputPathPre)
 	ytdlpPath := isYtDlpAvailable()
 	if ytdlpPath == "" {
@@ -225,16 +225,6 @@ func DownloadYouTubeVideo(url, outputPathPre, format, dType string, progressCh c
 		return fmt.Errorf("error executing yt-dlp: %v", err)
 	}
 
-	// Get file size after download is complete
-	var totalSizeBytes int64
-	fileInfo, err := os.Stat(outputPathPre) // won't always be the same because video can default to webm
-	if err == nil {
-		totalSizeBytes = fileInfo.Size()
-	} else {
-		totalSizeBytes = 1
-	}
-	progressCh <- totalSizeBytes
-
 	if musicId != "" {
 		// outputPathPre works here because audio is always m4a, but user can mess it up
 		_ = addMusicMetadata(outputPathPre, musicClient, musicId)

internal/downloader.go

@@ -164,6 +164,7 @@ func BatchDownload(entries []utils.DownloadEntry, numLinks, connectionsPerLink i
 				// YouTube download (with yt-dlp as dependency)
 				// =================================================================================================================
 				case "youtube":
+					close(progressCh) // Note needed for youtube downloads
 					processedURL, format, dType, output, err := danzoyoutube.ProcessURL(entry.URL)
 					if err != nil {
 						outputMgr.ReportError(entryFunctionId, fmt.Errorf("error processing YouTube URL %s: %v", entry.OutputPath, err))
@@ -199,9 +200,8 @@ func BatchDownload(entries []utils.DownloadEntry, numLinks, connectionsPerLink i
 						}
 					}(entry.OutputPath, streamCh)
 
-					err = danzoyoutube.DownloadYouTubeVideo(entry.URL, entry.OutputPath, format, dType, progressCh, streamCh)
+					err = danzoyoutube.DownloadYouTubeVideo(entry.URL, entry.OutputPath, format, dType, streamCh)
 					close(streamCh)
-					close(progressCh) // temporary; TODO: remove progress for YT or print size
 					if err != nil {
 						outputMgr.ReportError(entryFunctionId, fmt.Errorf("error downloading %s: %v", entry.OutputPath, err))
 						outputMgr.SetMessage(entryFunctionId, fmt.Sprintf("Error downloading %s", entry.OutputPath))