Skip to content

Commit 4a110ea

Browse files
committed
revert extraneous changes
1 parent 4be43b5 commit 4a110ea

2 files changed

Lines changed: 11 additions & 8 deletions

File tree

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
default: patch
3+
---
4+
5+
Fix crash when previewing non-video YouTube URLs (channels, @handles, etc.) that lack query parameters.

src/app/components/url-preview/ClientPreview.tsx

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -168,21 +168,19 @@ function parseYoutubeLink(url: string): YoutubeLink | null {
168168
[videoId] = split;
169169
params = split[1]?.split('&');
170170
} else if (url.includes('/shorts/')) {
171-
const split = path.split('/shorts/');
171+
const split = path.split('?');
172172
[videoId] = split;
173-
params = split[1]?.split('shorts');
173+
params = split[1]?.split('&') ?? [];
174174
} else if (url.includes('youtube.com')) {
175-
params = path.split('?')[1].split('&');
176-
videoId = params.find((s) => s.startsWith('v='), params)?.split('v=')[1];
175+
params = path.split('?')[1]?.split('&') ?? [];
176+
videoId = params.find((s) => s.startsWith('v='))?.split('v=')[1];
177177
} else return null;
178178

179179
if (!videoId) return null;
180180

181181
// playlist is not used for the embed, it can be appended as is
182-
const playlist = params ? params.find((s) => s.startsWith('list='), params) : undefined;
183-
const timestamp = params
184-
? params.find((s) => s.startsWith('t='), params)?.split('t=')[1]
185-
: undefined;
182+
const playlist = params ? params.find((s) => s.startsWith('list=')) : undefined;
183+
const timestamp = params ? params.find((s) => s.startsWith('t='))?.split('t=')[1] : undefined;
186184

187185
return {
188186
videoId,

0 commit comments

Comments
 (0)