Allow newIssueUrl to be specified by a hash query param

src/index.test.ts

@@ -16,11 +16,19 @@ const phishingHtml = readFileSync(
  *
  * @param hostname - The hostname to include in the URL fragment.
  * @param href - The href to include in the URL fragment.
+ * @param newIssueUrl - Optional, the url for the new issue link
+ * to include in in the URL fragment.
  * @returns The phishing warning page URL.
  */
-function getUrl(hostname?: string, href?: string) {
+function getUrl(hostname?: string, href?: string, newIssueUrl?: string) {
   const baseUrl = 'https://metamask.github.io/phishing-warning/#';
-  if (hostname && href) {
+  if (hostname && href && newIssueUrl) {
+    return `${baseUrl}hostname=${encodeURIComponent(
+      hostname,
+    )}&href=${encodeURIComponent(href)}&newIssueUrl=${encodeURIComponent(
+      newIssueUrl,
+    )}`;
+  } else if (hostname && href) {
     return `${baseUrl}hostname=${encodeURIComponent(
       hostname,
     )}&href=${encodeURIComponent(href)}`;
@@ -98,6 +106,21 @@ describe('Phishing warning page', () => {
     );
   });
 
+  it('should have the correct "New issue" link if a newIssueUrl is specified in the hash query string', async () => {
+    mockLocation(
+      getUrl('example.com', 'https://example.com', 'https://example.com'),
+    );
+
+    // non-null assertion used because TypeScript doesn't know the event handler was run
+    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+    onDomContentLoad!(new Event('DOMContentLoaded'));
+
+    const newIssueLink = window.document.getElementById('new-issue-link');
+    expect(newIssueLink?.getAttribute('href')).toBe(
+      'https://example.com?title=[Legitimate%20Site%20Blocked]%20example.com&body=https%3A%2F%2Fexample.com',
+    );
+  });
+
   it.todo(
     'should add site to safelist when the user continues at their own risk',
   );

src/index.ts

@@ -109,7 +109,9 @@ function start() {
     throw new Error('Unable to locate new issue link');
   }
 
-  const newIssueUrl = `https://github.com/MetaMask/eth-phishing-detect/issues/new`;
+  const newIssueUrl =
+    hashQueryString.get('newIssueUrl') ||
+    `https://github.com/MetaMask/eth-phishing-detect/issues/new`;
   const newIssueParams = `?title=[Legitimate%20Site%20Blocked]%20${encodeURIComponent(
     suspectHostname,
   )}&body=${encodeURIComponent(suspectHref)}`;