Remove Permalink Preview Feature Flag (#23731)
Co-authored-by: Mattermost Build <build@mattermost.com>
Этот коммит содержится в:
коммит произвёл
GitHub
родитель
2cec338915
Коммит
4e92dde5b3
@@ -292,7 +292,7 @@ func (a *App) getEmbedForPost(c request.CTX, post *model.Post, firstLink string,
|
|||||||
return nil, err
|
return nil, err
|
||||||
}
|
}
|
||||||
|
|
||||||
if !*a.Config().ServiceSettings.EnablePermalinkPreviews || !a.Config().FeatureFlags.PermalinkPreviews {
|
if !*a.Config().ServiceSettings.EnablePermalinkPreviews {
|
||||||
permalink = nil
|
permalink = nil
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -582,7 +582,7 @@ func (a *App) getLinkMetadata(c request.CTX, requestURL string, timestamp int64,
|
|||||||
|
|
||||||
// Check cache
|
// Check cache
|
||||||
og, image, permalink, ok := getLinkMetadataFromCache(requestURL, timestamp)
|
og, image, permalink, ok := getLinkMetadataFromCache(requestURL, timestamp)
|
||||||
if !*a.Config().ServiceSettings.EnablePermalinkPreviews || !a.Config().FeatureFlags.PermalinkPreviews {
|
if !*a.Config().ServiceSettings.EnablePermalinkPreviews {
|
||||||
permalink = nil
|
permalink = nil
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -600,7 +600,7 @@ func (a *App) getLinkMetadata(c request.CTX, requestURL string, timestamp int64,
|
|||||||
}
|
}
|
||||||
|
|
||||||
var err error
|
var err error
|
||||||
if looksLikeAPermalink(requestURL, a.GetSiteURL()) && *a.Config().ServiceSettings.EnablePermalinkPreviews && a.Config().FeatureFlags.PermalinkPreviews {
|
if looksLikeAPermalink(requestURL, a.GetSiteURL()) && *a.Config().ServiceSettings.EnablePermalinkPreviews {
|
||||||
referencedPostID := requestURL[len(requestURL)-26:]
|
referencedPostID := requestURL[len(requestURL)-26:]
|
||||||
|
|
||||||
referencedPost, appErr := a.GetSinglePost(referencedPostID, false)
|
referencedPost, appErr := a.GetSinglePost(referencedPostID, false)
|
||||||
|
|||||||
@@ -2505,7 +2505,6 @@ func TestGetLinkMetadata(t *testing.T) {
|
|||||||
th.App.UpdateConfig(func(cfg *model.Config) {
|
th.App.UpdateConfig(func(cfg *model.Config) {
|
||||||
*cfg.ServiceSettings.EnablePermalinkPreviews = true
|
*cfg.ServiceSettings.EnablePermalinkPreviews = true
|
||||||
*cfg.ServiceSettings.SiteURL = server.URL
|
*cfg.ServiceSettings.SiteURL = server.URL
|
||||||
cfg.FeatureFlags.PermalinkPreviews = true
|
|
||||||
})
|
})
|
||||||
|
|
||||||
requestURL := server.URL + "/pl/5rpoy4o3nbgwjm7gs4cm71h6ho"
|
requestURL := server.URL + "/pl/5rpoy4o3nbgwjm7gs4cm71h6ho"
|
||||||
|
|||||||
@@ -73,7 +73,6 @@ func (f *FeatureFlags) SetDefaults() {
|
|||||||
f.AppsEnabled = true
|
f.AppsEnabled = true
|
||||||
f.PluginApps = ""
|
f.PluginApps = ""
|
||||||
f.PluginFocalboard = ""
|
f.PluginFocalboard = ""
|
||||||
f.PermalinkPreviews = true
|
|
||||||
f.BoardsFeatureFlags = ""
|
f.BoardsFeatureFlags = ""
|
||||||
f.BoardsDataRetention = false
|
f.BoardsDataRetention = false
|
||||||
f.NormalizeLdapDNs = false
|
f.NormalizeLdapDNs = false
|
||||||
|
|||||||
Ссылка в новой задаче
Block a user