fix: improve error catching of injected user scripts by ACTCD · Pull Request #754 · quoid/userscripts · GitHub
Skip to content

fix: improve error catching of injected user scripts#754

Merged
ACTCD merged 2 commits into
mainfrom
inject-js-try-catch-improves
Nov 16, 2024
Merged

fix: improve error catching of injected user scripts#754
ACTCD merged 2 commits into
mainfrom
inject-js-try-catch-improves

Conversation

@ACTCD

@ACTCD ACTCD commented Nov 15, 2024

Copy link
Copy Markdown
Collaborator
  • Fix async error not caught introduced by feat: supports top-level await and namespace isolation #750

  • Fix errors may not show up in console when injecting page context

    • For example, the page script prevented the default event:
      window.addEventListener('unhandledrejection', (event) => {
          event.stopImmediatePropagation();
          event.preventDefault();
      });
  • Fix the sourceURL resource not show up when the script name contains quotation marks

@ACTCD ACTCD requested a review from quoid November 15, 2024 23:16
@ACTCD ACTCD merged commit ddead3d into main Nov 16, 2024
@ACTCD ACTCD deleted the inject-js-try-catch-improves branch November 16, 2024 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants