status-im
/
status-go
mirror of https://github.com/status-im/status-go.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

Comment out Medium unfurling (#2193)

This commit is contained in:
Shivek Khurana 2021-04-12 14:49:34 +05:30 committed by GitHub
parent 27f5ad23cf
commit 9ef4971fe1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 22 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
protocol/urls/urls.go
View File

@ -99,11 +99,13 @@ func LinkPreviewWhitelist() []Site {
Address: "github.com",
ImageSite: false,
},
Site{
Title: "Medium",
Address: "medium.com",
ImageSite: false,
},
// Medium unfurling is failing - https://github.com/status-im/status-go/issues/2192
//
// Site{
// Title: "Medium",
// Address: "medium.com",
// ImageSite: false,
// },
}
}
@ -266,7 +268,7 @@ func GetLinkPreviewData(link string) (previewData LinkPreviewData, err error) {
switch hostname {
case "youtube.com", "youtu.be", "www.youtube.com":
return GetYoutubePreviewData(link)
case "github.com", "our.status.im", "medium.com":
case "github.com", "our.status.im":
return GetGenericLinkPreviewData(link)
case "giphy.com", "media.giphy.com":
return GetGiphyPreviewData(link)

26
protocol/urls/urls_test.go
View File

@ -135,17 +135,19 @@ func TestStatusLinkPreviewData(t *testing.T) {
require.Equal(t, statusSecurityAudit.ThumbnailURL, previewData.ThumbnailURL)
}
func TestMediumLinkPreviewData(t *testing.T) {
// Medium unfurling is failing - https://github.com/status-im/status-go/issues/2192
//
// func TestMediumLinkPreviewData(t *testing.T) {
statusSecurityAudit := LinkPreviewData{
Site: "Medium",
Title: "A Look at the Status.im ICO Token Distribution",
ThumbnailURL: "https://miro.medium.com/max/700/1*Smc0y_TOL1XsofS1wxa3rg.jpeg",
}
// statusSecurityAudit := LinkPreviewData{
// Site: "Medium",
// Title: "A Look at the Status.im ICO Token Distribution",
// ThumbnailURL: "https://miro.medium.com/max/700/1*Smc0y_TOL1XsofS1wxa3rg.jpeg",
// }
previewData, err := GetLinkPreviewData("https://medium.com/the-bitcoin-podcast-blog/a-look-at-the-status-im-ico-token-distribution-f5bcf7f00907")
require.NoError(t, err)
require.Equal(t, statusSecurityAudit.Site, previewData.Site)
require.Equal(t, statusSecurityAudit.Title, previewData.Title)
require.Equal(t, statusSecurityAudit.ThumbnailURL, previewData.ThumbnailURL)
}
// previewData, err := GetLinkPreviewData("https://medium.com/the-bitcoin-podcast-blog/a-look-at-the-status-im-ico-token-distribution-f5bcf7f00907")
// require.NoError(t, err)
// require.Equal(t, statusSecurityAudit.Site, previewData.Site)
// require.Equal(t, statusSecurityAudit.Title, previewData.Title)
// require.Equal(t, statusSecurityAudit.ThumbnailURL, previewData.ThumbnailURL)
// }