Merge pull request #1467 from omnivore-app/fix/hide-url-check

Handle invalid URLs in the shouldHideUrl check
This commit is contained in:
Jackson Harper
2022-11-25 16:31:11 +08:00
committed by GitHub

View File

@ -5,10 +5,14 @@ import { HighlightItemCard } from './HighlightItemCard'
import { PageType } from '../../../lib/networking/fragments/articleFragment'
const shouldHideUrl = (url: string): boolean => {
const origin = new URL(url).origin
const hideHosts = ['https://storage.googleapis.com', 'https://omnivore.app']
if (hideHosts.indexOf(origin) != -1) {
return true
try {
const origin = new URL(url).origin
const hideHosts = ['https://storage.googleapis.com', 'https://omnivore.app']
if (hideHosts.indexOf(origin) != -1) {
return true
}
} catch {
console.log('invalid url item', url)
}
return false
}