fix: remove tenor.com from preview whitelist

This commit is contained in:
Stefan 2022-09-08 15:50:17 +01:00 committed by Stefan Dunca
parent d689b337c9
commit bcdb14bd48
1 changed files with 4 additions and 9 deletions

View File

@ -101,15 +101,10 @@ func LinkPreviewWhitelist() []Site {
ImageSite: false, ImageSite: false,
}, },
{ {
Title: "tenor GIFs subdomain", Title: "Tenor GIFs subdomain",
Address: "media.tenor.com", Address: "media.tenor.com",
ImageSite: false, ImageSite: false,
}, },
{
Title: "tenor GIFs",
Address: "tenor.com",
ImageSite: false,
},
// Medium unfurling is failing - https://github.com/status-im/status-go/issues/2192 // Medium unfurling is failing - https://github.com/status-im/status-go/issues/2192
// //
// { // {
@ -192,7 +187,7 @@ func GetGenericLinkPreviewData(link string) (previewData LinkPreviewData, err er
return previewData, nil return previewData, nil
} }
func GetGenericImageLinkPreviewData(title string, link string) (previewData LinkPreviewData, err error) { func FakeGenericImageLinkPreviewData(title string, link string) (previewData LinkPreviewData, err error) {
u, _ := url.Parse(link) u, _ := url.Parse(link)
previewData.Title = title previewData.Title = title
previewData.Site = strings.ToLower(u.Hostname()) previewData.Site = strings.ToLower(u.Hostname())
@ -263,8 +258,8 @@ func GetLinkPreviewData(link string) (previewData LinkPreviewData, err error) {
return GetGiphyShortURLPreviewData(link) return GetGiphyShortURLPreviewData(link)
case "twitter.com": case "twitter.com":
return GetTwitterPreviewData(link) return GetTwitterPreviewData(link)
case "media.tenor.com", "tenor.com": case "media.tenor.com":
return GetGenericImageLinkPreviewData("Tenor", link) return FakeGenericImageLinkPreviewData("Tenor", link)
default: default:
return previewData, fmt.Errorf("link %s isn't whitelisted. Hostname - %s", link, u.Hostname()) return previewData, fmt.Errorf("link %s isn't whitelisted. Hostname - %s", link, u.Hostname())
} }