mirrors/mostlymatter
1
0
Форкнуть 0
Код Задачи Запросы на слияние Действия Пакеты Проекты Релизы Вики Активность

[MM-40560] Add Server locale to link preview request (#19156)

Просмотр исходного кода 
* Add Server locale to link preview request

* Update unit tests for post embed
Этот коммит содержится в:
Ben
2022-01-05 17:32:36 -05:00
коммит произвёл GitHub
родитель f4809def78
Коммит 9ba020db2e
2 изменённых файлов: 39 добавлений и 8 удалений
Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

1
app/post_metadata.go
Просмотреть файл

@@ -549,6 +549,7 @@ func (a *App) getLinkMetadata(requestURL string, timestamp int64, isNewPost bool
} else { } else {
request.Header.Add("Accept", "image/*") request.Header.Add("Accept", "image/*")
request.Header.Add("Accept", "text/html;q=0.8") request.Header.Add("Accept", "text/html;q=0.8")
request.Header.Add("Accept-Language", *a.Config().LocalizationSettings.DefaultServerLocale)
client := a.HTTPService().MakeClient(false) client := a.HTTPService().MakeClient(false)
client.Timeout = time.Duration(*a.Config().ExperimentalSettings.LinkMetadataTimeoutMilliseconds) * time.Millisecond client.Timeout = time.Duration(*a.Config().ExperimentalSettings.LinkMetadataTimeoutMilliseconds) * time.Millisecond

46
app/post_metadata_test.go
Просмотреть файл

@@ -754,12 +754,24 @@ func TestGetEmbedForPost(t *testing.T) {
server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
if r.URL.Path == "/index.html" { if r.URL.Path == "/index.html" {
w.Header().Set("Content-Type", "text/html") w.Header().Set("Content-Type", "text/html")
w.Write([]byte(` if r.Header.Get("Accept-Language") == "fr" {
<html> w.Header().Set("Content-Language", "fr")
<head> w.Write([]byte(`
<meta property="og:title" content="Title" /> <html>
</head> <head>
</html>`)) <meta property="og:title" content="Title-FR" />
<meta property="og:description" content="Bonjour le monde" />
</head>
</html>`))
} else {
w.Write([]byte(`
<html>
<head>
<meta property="og:title" content="Title" />
<meta property="og:description" content="Hello world" />
</head>
</html>`))
}
} else if r.URL.Path == "/image.png" { } else if r.URL.Path == "/image.png" {
file, err := testutils.ReadTestFile("test.png") file, err := testutils.ReadTestFile("test.png")
require.NoError(t, err) require.NoError(t, err)
@@ -819,14 +831,32 @@ func TestGetEmbedForPost(t *testing.T) {
assert.NoError(t, err) assert.NoError(t, err)
}) })
t.Run("should return an image embed when the first link is an image", func(t *testing.T) { t.Run("should return an opengraph embed", func(t *testing.T) {
embed, err := th.App.getEmbedForPost(&model.Post{}, ogURL, false) embed, err := th.App.getEmbedForPost(&model.Post{}, ogURL, false)
assert.Equal(t, &model.PostEmbed{ assert.Equal(t, &model.PostEmbed{
Type: model.PostEmbedOpengraph, Type: model.PostEmbedOpengraph,
URL: ogURL, URL: ogURL,
Data: &opengraph.OpenGraph{ Data: &opengraph.OpenGraph{
Title: "Title", Title: "Title",
Description: "Hello world",
},
}, embed)
assert.NoError(t, err)
})
t.Run("should return an opengraph embed in different Server Language", func(t *testing.T) {
th.App.UpdateConfig(func(cfg *model.Config) {
*cfg.LocalizationSettings.DefaultServerLocale = "fr"
})
embed, err := th.App.getEmbedForPost(&model.Post{}, ogURL, false)
assert.Equal(t, &model.PostEmbed{
Type: model.PostEmbedOpengraph,
URL: ogURL,
Data: &opengraph.OpenGraph{
Title: "Title-FR",
Description: "Bonjour le monde",
}, },
}, embed) }, embed)
assert.NoError(t, err) assert.NoError(t, err)