heimoshuiyu/gonic
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fix(jukebox): gracefully handle case of no audio in feed item

Browse Source 
fixes #269
This commit is contained in:
sentriz
2022-12-12 18:30:40 +00:00
parent 2bc3f31554
commit b47c880ea5
1 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
podcasts/podcasts.go
View File

@@ -24,6 +24,8 @@ import (
"go.senan.xyz/gonic/scanner/tags" "go.senan.xyz/gonic/scanner/tags"
) )
var ErrNoAudioInFeedItem = errors.New("no audio in feed item")
const downloadAllWaitInterval = 3 * time.Second const downloadAllWaitInterval = 3 * time.Second
const fetchUserAgent = `Mozilla/5.0 (Macintosh; Intel Mac OS X 10_7_5) AppleWebKit/537.11 (KHTML, like Gecko) Chrome/23.0.1271.64 Safari/537.11` const fetchUserAgent = `Mozilla/5.0 (Macintosh; Intel Mac OS X 10_7_5) AppleWebKit/537.11 (KHTML, like Gecko) Chrome/23.0.1271.64 Safari/537.11`
@@ -166,6 +168,10 @@ func (p *Podcasts) AddNewEpisodes(podcast *db.Podcast, items []*gofeed.Item) err
} }
for _, item := range getEntriesAfterDate(items, *podcastEpisode.PublishDate) { for _, item := range getEntriesAfterDate(items, *podcastEpisode.PublishDate) {
episode, err := p.AddEpisode(podcast.ID, item) episode, err := p.AddEpisode(podcast.ID, item)
if errors.Is(err, ErrNoAudioInFeedItem) {
log.Println("failed to find audio in feed item, skipping")
continue
}
if err != nil { if err != nil {
return err return err
} }
@@ -226,9 +232,7 @@ func (p *Podcasts) AddEpisode(podcastID int, item *gofeed.Item) (*db.PodcastEpis
} }
return episode, nil return episode, nil
} }
// hopefully shouldnt reach here return nil, ErrNoAudioInFeedItem
log.Println("failed to find audio in feed item, skipping")
return nil, nil
} }
func isAudio(mediaType, url string) bool { func isAudio(mediaType, url string) bool {