diff options
author | Ajay <[email protected]> | 2024-09-08 01:20:25 -0400 |
---|---|---|
committer | Ajay <[email protected]> | 2024-09-08 01:20:25 -0400 |
commit | 12e8c7aad8392039b641c0eb414cd22e6b80ecb7 (patch) | |
tree | 2079562548e820de18cc5db85f827b7e510205a8 | |
parent | 2a2896786cba795aa2f5b83b8cfcd75842fc2c02 (diff) | |
download | SponsorBlock-12e8c7aad8392039b641c0eb414cd22e6b80ecb7.tar.gz SponsorBlock-12e8c7aad8392039b641c0eb414cd22e6b80ecb7.zip |
Add way to copy debug logs from popup
m--------- | public/_locales | 0 | ||||
-rw-r--r-- | public/popup.html | 2 | ||||
-rw-r--r-- | src/config.ts | 24 | ||||
-rw-r--r-- | src/content.ts | 6 | ||||
-rw-r--r-- | src/messageTypes.ts | 11 | ||||
-rw-r--r-- | src/options.ts | 30 | ||||
-rw-r--r-- | src/popup.ts | 13 |
7 files changed, 57 insertions, 29 deletions
diff --git a/public/_locales b/public/_locales -Subproject 7cedce215851872be28bd50a9f690b680dd5444 +Subproject 3c5a80ca9cc600c7c03b8d513cd80df8aa17b24 diff --git a/public/popup.html b/public/popup.html index bee090c4..2b5034f7 100644 --- a/public/popup.html +++ b/public/popup.html @@ -196,6 +196,8 @@ <a href="https://discord.gg/SponsorBlock" target="_blank" rel="noopener">Discord</a> <a href="https://matrix.to/#/#sponsor:ajay.app?via=ajay.app&via=matrix.org&via=mozilla.org" target="_blank" rel="noopener">Matrix</a> <a href="https://sponsor.ajay.app/donate" target="_blank" rel="noopener" id="sbDonate">__MSG_Donate__</a> + <br /> + <a id="debugLogs">__MSG_copyDebugLogs__</a> </footer> <button id="showNoticeAgain" style="display: none">__MSG_showNotice__</button> diff --git a/src/config.ts b/src/config.ts index 3fbe23ba..383bfcf6 100644 --- a/src/config.ts +++ b/src/config.ts @@ -477,4 +477,26 @@ const localDefaults = { }; const Config = new ConfigClass(syncDefaults, localDefaults, migrateOldSyncFormats); -export default Config;
\ No newline at end of file +export default Config; + +export function generateDebugDetails(): string { + // Build output debug information object + const output = { + debug: { + userAgent: navigator.userAgent, + platform: navigator.platform, + language: navigator.language, + extensionVersion: chrome.runtime.getManifest().version + }, + config: JSON.parse(JSON.stringify(Config.cachedSyncConfig)) // Deep clone config object + }; + + // Sanitise sensitive user config values + delete output.config.userID; + output.config.serverAddress = (output.config.serverAddress === CompileConfig.serverAddress) + ? "Default server address" : "Custom server address"; + output.config.invidiousInstances = output.config.invidiousInstances.length; + output.config.whitelistedChannels = output.config.whitelistedChannels.length; + + return JSON.stringify(output, null, 4); +}
\ No newline at end of file diff --git a/src/content.ts b/src/content.ts index 398102aa..04c2dcd3 100644 --- a/src/content.ts +++ b/src/content.ts @@ -346,6 +346,12 @@ function messageListener(request: Message, sender: unknown, sendResponse: (respo metaKey: request.metaKey })); break; + case "getLogs": + sendResponse({ + debug: window["SBLogs"].debug, + warn: window["SBLogs"].warn + }); + break; } sendResponse({}); diff --git a/src/messageTypes.ts b/src/messageTypes.ts index 191166a0..6b9f3225 100644 --- a/src/messageTypes.ts +++ b/src/messageTypes.ts @@ -17,7 +17,8 @@ interface DefaultMessage { | "isChannelWhitelisted" | "submitTimes" | "refreshSegments" - | "closePopup"; + | "closePopup" + | "getLogs"; } interface BoolValueMessage { @@ -104,7 +105,8 @@ export type MessageResponse = | Record<string, never> // empty object response {} | VoteResponse | ImportSegmentsResponse - | RefreshSegmentsResponse; + | RefreshSegmentsResponse + | LogResponse; export interface VoteResponse { successType: number; @@ -120,6 +122,11 @@ export interface RefreshSegmentsResponse { hasVideo: boolean; } +export interface LogResponse { + debug: string[]; + warn: string[]; +} + export interface TimeUpdateMessage { message: "time"; time: number; diff --git a/src/options.ts b/src/options.ts index e983498f..f6ba2663 100644 --- a/src/options.ts +++ b/src/options.ts @@ -1,7 +1,7 @@ import * as React from "react"; import { createRoot } from 'react-dom/client'; -import Config from "./config"; +import Config, { generateDebugDetails } from "./config"; import * as CompileConfig from "../config.json"; import * as invidiousList from "../ci/invidiouslist.json"; @@ -698,32 +698,14 @@ function validateServerAddress(input: string): string { } function copyDebugOutputToClipboard() { - // Build output debug information object - const output = { - debug: { - userAgent: navigator.userAgent, - platform: navigator.platform, - language: navigator.language, - extensionVersion: chrome.runtime.getManifest().version - }, - config: JSON.parse(JSON.stringify(Config.cachedSyncConfig)) // Deep clone config object - }; - - // Sanitise sensitive user config values - delete output.config.userID; - output.config.serverAddress = (output.config.serverAddress === CompileConfig.serverAddress) - ? "Default server address" : "Custom server address"; - output.config.invidiousInstances = output.config.invidiousInstances.length; - output.config.whitelistedChannels = output.config.whitelistedChannels.length; - // Copy object to clipboard - navigator.clipboard.writeText(JSON.stringify(output, null, 4)) - .then(() => { + navigator.clipboard.writeText(generateDebugDetails()) + .then(() => { alert(chrome.i18n.getMessage("copyDebugInformationComplete")); - }) - .catch(() => { + }) + .catch(() => { alert(chrome.i18n.getMessage("copyDebugInformationFailed")); - }); + }); } function isIncognitoAllowed(): Promise<boolean> { diff --git a/src/popup.ts b/src/popup.ts index 27984cee..b20723a2 100644 --- a/src/popup.ts +++ b/src/popup.ts @@ -1,4 +1,4 @@ -import Config from "./config"; +import Config, { generateDebugDetails } from "./config"; import Utils from "./utils"; import { @@ -12,6 +12,7 @@ import { GetChannelIDResponse, IsChannelWhitelistedResponse, IsInfoFoundMessageResponse, + LogResponse, Message, MessageResponse, PopupMessage, @@ -184,7 +185,8 @@ async function runThePopup(messageListener?: MessageListener): Promise<void> { "exportSegmentsButton", "importSegmentsMenu", "importSegmentsText", - "importSegmentsSubmit" + "importSegmentsSubmit", + "debugLogs" ].forEach(id => PageElements[id] = document.getElementById(id)); @@ -255,6 +257,7 @@ async function runThePopup(messageListener?: MessageListener): Promise<void> { PageElements.helpButton.addEventListener("click", openHelp); PageElements.refreshSegmentsButton.addEventListener("click", refreshSegments); PageElements.sbPopupIconCopyUserID.addEventListener("click", async () => copyToClipboard(await getHash(Config.config.userID))); + PageElements.debugLogs.addEventListener("click", copyDebgLogs); // Forward click events if (window !== window.top) { @@ -1143,6 +1146,12 @@ async function runThePopup(messageListener?: MessageListener): Promise<void> { } } + function copyDebgLogs() { + sendTabMessage({ message: "getLogs" }, (logs: LogResponse) => { + copyToClipboard(`${generateDebugDetails()}\n\nWarn:\n${logs.warn.join("\n")}\n\nDebug:\n${logs.debug.join("\n")}`); + }); + } + function onMessage(msg: PopupMessage) { switch (msg.message) { case "time": |