ignore video if video.ContentDetails.Duration is empty, usually upcoming live broadcast.

Author: fqx

pkg/builder/youtube.go

@@ -317,19 +317,16 @@ func (yt *YouTubeBuilder) queryVideoDescriptions(ctx context.Context, playlist m
 
 			// Sometimes YouTube retrun empty content defailt, use arbitrary one
 			var seconds int64 = 1
-			if video.ContentDetails != nil {
+			if video.ContentDetails.Duration != "" {
 				// Parse duration
 				d, err := duration.FromString(video.ContentDetails.Duration)
 				if err != nil {
-				    // If parsing fails, log the error and use the default value
-				    log.Printf("Warning: Failed to parse duration %s: %v. Using default value.", video.ContentDetails.Duration, err)
-				    // seconds is already set to 1, so we don't need to set it again
-				} else {
-				    // If parsing succeeds, set seconds to the parsed duration
-				    seconds = int64(d.ToDuration().Seconds())
+					return errors.Wrapf(err, "failed to parse duration %s", video.ContentDetails.Duration)
 				}
 
-				// seconds = int64(d.ToDuration().Seconds())
+				seconds = int64(d.ToDuration().Seconds())
+			} else {
+				continue
 			}
 
 			var (