yetangitu/Peertube
1
0
Fork 0
mirror of https://github.com/Chocobozzz/PeerTube.git synced 2025-10-03 09:49:20 +02:00
Code Issues Releases Wiki Activity

Prefer og:image for opengraph

Browse source 
See https://github.com/Chocobozzz/PeerTube/issues/7109
This commit is contained in:
Chocobozzz 2025-06-27 10:00:51 +02:00
parent 53472daa07
commit 6c85bbf852
No known key found for this signature in database
GPG key ID: 583A612D890159BE
2 changed files with 6 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

10
packages/tests/src/client/head-tags.ts
View file

@ -86,7 +86,7 @@ describe('Test <head> HTML tags', function () {
expect(text).to.contain(`<meta property="og:description" content="${instanceConfig.shortDescription}" />`) expect(text).to.contain(`<meta property="og:description" content="${instanceConfig.shortDescription}" />`)
expect(text).to.contain('<meta property="og:type" content="website" />') expect(text).to.contain('<meta property="og:type" content="website" />')
expect(text).to.contain(`<meta property="og:url" content="${url}`) expect(text).to.contain(`<meta property="og:url" content="${url}`)
expect(text).to.contain(`<meta property="og:image:url" content="${servers[0].url}/`) expect(text).to.contain(`<meta property="og:image" content="${servers[0].url}/`)
} }
async function accountPageTest (path: string) { async function accountPageTest (path: string) {
@ -97,7 +97,7 @@ describe('Test <head> HTML tags', function () {
expect(text).to.contain(`<meta property="og:description" content="${account.description}" />`) expect(text).to.contain(`<meta property="og:description" content="${account.description}" />`)
expect(text).to.contain('<meta property="og:type" content="website" />') expect(text).to.contain('<meta property="og:type" content="website" />')
expect(text).to.contain(`<meta property="og:url" content="${servers[0].url}/a/${servers[0].store.user.username}/video-channels" />`) expect(text).to.contain(`<meta property="og:url" content="${servers[0].url}/a/${servers[0].store.user.username}/video-channels" />`)
expect(text).to.not.contain(`<meta property="og:image:url"`) expect(text).to.not.contain(`<meta property="og:image"`)
} }
async function channelPageTest (path: string) { async function channelPageTest (path: string) {
@ -108,7 +108,7 @@ describe('Test <head> HTML tags', function () {
expect(text).to.contain(`<meta property="og:description" content="${channelDescription}" />`) expect(text).to.contain(`<meta property="og:description" content="${channelDescription}" />`)
expect(text).to.contain('<meta property="og:type" content="website" />') expect(text).to.contain('<meta property="og:type" content="website" />')
expect(text).to.contain(`<meta property="og:url" content="${servers[0].url}/c/${servers[0].store.channel.name}/videos" />`) expect(text).to.contain(`<meta property="og:url" content="${servers[0].url}/c/${servers[0].store.channel.name}/videos" />`)
expect(text).to.contain(`<meta property="og:image:url" content="${servers[0].url}/`) expect(text).to.contain(`<meta property="og:image" content="${servers[0].url}/`)
} }
async function watchVideoPageTest (path: string) { async function watchVideoPageTest (path: string) {
@ -119,7 +119,7 @@ describe('Test <head> HTML tags', function () {
expect(text).to.contain(`<meta property="og:description" content="${videoDescriptionPlainText}" />`) expect(text).to.contain(`<meta property="og:description" content="${videoDescriptionPlainText}" />`)
expect(text).to.contain('<meta property="og:type" content="video" />') expect(text).to.contain('<meta property="og:type" content="video" />')
expect(text).to.contain(`<meta property="og:url" content="${servers[0].url}/w/${servers[0].store.video.shortUUID}" />`) expect(text).to.contain(`<meta property="og:url" content="${servers[0].url}/w/${servers[0].store.video.shortUUID}" />`)
expect(text).to.contain(`<meta property="og:image:url" content="${servers[0].url}/`) expect(text).to.contain(`<meta property="og:image" content="${servers[0].url}/`)
} }
async function watchPlaylistPageTest (path: string) { async function watchPlaylistPageTest (path: string) {
@ -130,7 +130,7 @@ describe('Test <head> HTML tags', function () {
expect(text).to.contain(`<meta property="og:description" content="${playlistDescription}" />`) expect(text).to.contain(`<meta property="og:description" content="${playlistDescription}" />`)
expect(text).to.contain('<meta property="og:type" content="video" />') expect(text).to.contain('<meta property="og:type" content="video" />')
expect(text).to.contain(`<meta property="og:url" content="${servers[0].url}/w/p/${playlist.shortUUID}" />`) expect(text).to.contain(`<meta property="og:url" content="${servers[0].url}/w/p/${playlist.shortUUID}" />`)
expect(text).to.contain(`<meta property="og:image:url" content="${servers[0].url}/`) expect(text).to.contain(`<meta property="og:image" content="${servers[0].url}/`)
} }
it('Should have valid Open Graph tags on the common page', async function () { it('Should have valid Open Graph tags on the common page', async function () {

2
server/core/lib/html/shared/tags-html.ts
View file

@ -170,7 +170,7 @@ export class TagsHtml {
} }
if (tags.image?.url) { if (tags.image?.url) {
metaTags['og:image:url'] = tags.image.url metaTags['og:image'] = tags.image.url
} }
if (tags.image?.width && tags.image?.height) { if (tags.image?.width && tags.image?.height) {