Merge pull request #4040 from thelounge/xpaw/media-embed
Skip video/audio embeds if og:type exists but does not specify it
This commit is contained in:
commit
4307d2da9d
@ -131,9 +131,23 @@ function parseHtmlMedia($, preview, client) {
|
|||||||
return new Promise((resolve, reject) => {
|
return new Promise((resolve, reject) => {
|
||||||
if (Helper.config.disableMediaPreview) {
|
if (Helper.config.disableMediaPreview) {
|
||||||
reject();
|
reject();
|
||||||
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
let foundMedia = false;
|
let foundMedia = false;
|
||||||
|
const openGraphType = $('meta[property="og:type"]').attr("content");
|
||||||
|
|
||||||
|
// Certain news websites may include video and audio tags,
|
||||||
|
// despite actually being an article (as indicated by og:type).
|
||||||
|
// If there is og:type tag, we will only select video or audio if it matches
|
||||||
|
if (
|
||||||
|
openGraphType &&
|
||||||
|
!openGraphType.startsWith("video") &&
|
||||||
|
!openGraphType.startsWith("music")
|
||||||
|
) {
|
||||||
|
reject();
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
["video", "audio"].forEach((type) => {
|
["video", "audio"].forEach((type) => {
|
||||||
if (foundMedia) {
|
if (foundMedia) {
|
||||||
|
Loading…
Reference in New Issue
Block a user