ekzyis/stacker.news
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

recognize all youtube links

Browse Source
This commit is contained in:
keyan 2023-01-11 14:58:52 -06:00
parent a10e4aa637
commit 15c76aa134
2 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
api/resolvers/item.js
View File

@ -472,13 +472,13 @@ export default {
let similar = `(http(s)?://)?${uri}/?` let similar = `(http(s)?://)?${uri}/?`
const whitelist = ['news.ycombinator.com/item', 'bitcointalk.org/index.php'] const whitelist = ['news.ycombinator.com/item', 'bitcointalk.org/index.php']
const youtube = ['www.youtube.com', 'youtu.be'] const youtube = ['www.youtube.com', 'youtube.com', 'youtu.be']
if (whitelist.includes(uri)) { if (whitelist.includes(uri)) {
similar += `\\${urlObj.search}` similar += `\\${urlObj.search}`
} else if (youtube.includes(urlObj.hostname)) { } else if (youtube.includes(urlObj.hostname)) {
// extract id and create both links // extract id and create both links
const matches = url.match(/(https?:\/\/)?((www\.)?(youtube(-nocookie)?|youtube.googleapis)\.com.*(v\/|v=|vi=|vi\/|e\/|embed\/|user\/.*\/u\/\d+\/)|youtu\.be\/)(?<id>[_0-9a-z-]+)/i) const matches = url.match(/(https?:\/\/)?((www\.)?(youtube(-nocookie)?|youtube.googleapis)\.com.*(v\/|v=|vi=|vi\/|e\/|embed\/|user\/.*\/u\/\d+\/)|youtu\.be\/)(?<id>[_0-9a-z-]+)/i)
similar = `(http(s)?://)?(www.youtube.com/watch\\?v=${matches?.groups?.id}|youtu.be/${matches?.groups?.id})` similar = `(http(s)?://)?((www.)?youtube.com/watch\\?v=${matches?.groups?.id}|youtu.be/${matches?.groups?.id})`
} else { } else {
similar += '((\\?|#)%)?' similar += '((\\?|#)%)?'
} }

5
lib/url.js
View File

@ -5,14 +5,13 @@ export function ensureProtocol (value) {
return value return value
} }
export function removeTracking (value) { export function removeTracking (value) {
const exprs = [ const exprs = [
// twitter URLs // twitter URLs
/^(?<url>https?:\/\/twitter\.com\/(?:#!\/)?(?<user>\w+)\/status(?:es)?\/(?<id>\d+))/, /^(?<url>https?:\/\/twitter\.com\/(?:#!\/)?(?<user>\w+)\/status(?:es)?\/(?<id>\d+))/
] ]
for (const expr of exprs) { for (const expr of exprs) {
value = expr.exec(value)?.groups.url ?? value; value = expr.exec(value)?.groups.url ?? value
} }
return value return value
} }