Merge pull request #139 from DeepDoge/master

Little clean up and fixes

src/modules/crypto/index.ts

@@ -70,22 +70,18 @@ export function resetProfileSettings() {
 
 async function apiRequest<T extends object>(method: 'GET' | 'POST', pathname: string, data: T) {
     const settings = await getExtensionSettingsAsync()
-    /* const urlResolverSettings = ytUrlResolversSettings[settings.urlResolver]
-    if (!urlResolverSettings.signRequest) throw new Error() */
 
-    const url = new URL(ytUrlResolversSettings.madiatorFinder.href/* urlResolverSettings.href */)
+    const url = new URL(ytUrlResolversSettings.madiatorFinder.href)
     url.pathname = path.join(url.pathname, pathname)
     url.searchParams.set('data', JSON.stringify(data))
 
-    if (true/* requiresSignature */) {
-        if (!settings.privateKey || !settings.publicKey)
-            throw new Error('There is no profile.')
+    if (!settings.privateKey || !settings.publicKey)
+        throw new Error('There is no profile.')
 
-        url.searchParams.set('keys', JSON.stringify({
-            signature: await sign(url.searchParams.toString(), settings.privateKey!),
-            publicKey: settings.publicKey
-        }))
-    }
+    url.searchParams.set('keys', JSON.stringify({
+        signature: await sign(url.searchParams.toString(), settings.privateKey!),
+        publicKey: settings.publicKey
+    }))
 
     const respond = await fetch(url.href, { method })
 

src/modules/yt/urlCache.ts

@@ -75,7 +75,6 @@ async function get(id: string): Promise<string | null | undefined> {
         await clearExpired()
         return undefined
     }
-    console.log('cache found', id, response.value)
     return response.value
 }
 

src/pages/popup/style.css

@@ -1,6 +1,5 @@
 #popup {
     width: 40em;
-    max-width: 100%;
     overflow: hidden;
     margin: auto;
 }
@@ -28,18 +27,12 @@ section {
     gap: .75em;
 }
 
-section>label {
+label {
     font-size: 1.75em;
     font-weight: bold;
     text-align: center;
 }
 
-section>b {
-    display: block;
-    width: 100%;
-    text-align: left;
-}
-
 section>* {
     padding: 0 1rem;
 }

src/scripts/background.ts

@@ -20,7 +20,6 @@ chrome.runtime.onMessage.addListener(({ method, data }, sender, sendResponse) =>
           const params: Parameters<typeof resolveById> = JSON.parse(data)
           // Don't create a new Promise for same ID until on going one is over.
           const promise = onGoingLbryPathnameRequest[data] ?? (onGoingLbryPathnameRequest[data] = resolveById(...params))
-          console.log('lbrypathname request', params, await promise)
           resolve(await promise)
         } catch (error) {
           sendResponse(`error: ${(error as any).toString()}`)

src/scripts/ytContent.tsx

@@ -231,7 +231,6 @@ import { getExtensionSettingsAsync, getSourcePlatfromSettingsFromHostname, getTa
       console.error("Background error on:", params)
       throw new Error(`Background error. ${response ?? ''}`)
     }
-    console.log(response)
     return response ? JSON.parse(response) : null
   }