LBRYFoundation/Watch-on-LBRY
1
0
Fork 0
mirror of https://github.com/LBRYFoundation/Watch-on-LBRY.git synced 2025-08-23 17:47:26 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

🍘 removed memory cache for ctxFromURL

Browse source 
- we already tell browser to cache the request
- and we dont need the response in the same call stack
This commit is contained in:
Shiba 2022-01-07 22:01:30 +00:00
parent 7c50daf7fc
commit c2aacaf307
1 changed files with 1 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/scripts/tabOnUpdated.ts
View file

@ -16,8 +16,6 @@ async function resolveYT(descriptor: YtIdResolverDescriptor) {
return segments.join('/');
}
const lbryPathnameCache: Record<string, string | undefined> = {};
async function ctxFromURL(href: string): Promise<UpdateContext | void> {
if (!href) return;
@ -29,8 +27,7 @@ async function ctxFromURL(href: string): Promise<UpdateContext | void> {
const descriptor = ytService.getId(href);
if (!descriptor) return; // couldn't get the ID, so we're done
const res = href in lbryPathnameCache ? lbryPathnameCache[href] : await resolveYT(descriptor);
lbryPathnameCache[href] = res;
const res = await resolveYT(descriptor); // NOTE: API call cached by the browser
if (!res) return; // couldn't find it on lbry, so we're done
return { descriptor, lbryPathname: res, redirect, targetPlatform };