Merge pull request #320 from luisfavila/fix-url-cleaning

This commit is contained in:
Max Isom 2021-09-14 14:20:01 -04:00 committed by GitHub
commit c97206bb8b
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 20 additions and 10 deletions

View file

@ -9,6 +9,7 @@ import pLimit from 'p-limit';
import shuffle from 'array-shuffle';
import {QueuedSong, QueuedPlaylist} from '../services/player';
import {TYPES} from '../types';
import {cleanUrl} from '../utils/url';
@injectable()
export default class {
@ -34,16 +35,7 @@ export default class {
async youtubeVideo(url: string): Promise<QueuedSong|null> {
try {
// Clean URL
const u = new URL(url);
for (const [name] of u.searchParams) {
if (name !== 'v') {
u.searchParams.delete(name);
}
}
const videoDetails = await this.youtube.videos.get(u.toString());
const videoDetails = await this.youtube.videos.get(cleanUrl(url));
return {
title: videoDetails.snippet.title,

18
src/utils/url.ts Normal file
View file

@ -0,0 +1,18 @@
import {URL} from 'url';
export const cleanUrl = (url: string) => {
try {
// Clean URL
const u = new URL(url);
for (const [name] of u.searchParams) {
if (name !== 'v') {
u.searchParams.delete(name);
}
}
return u.toString();
} catch (_: unknown) {
return url;
}
};