scriptoriumadmin/zotero
1
0
Fork 0
mirror of https://github.com/zotero/zotero.git synced 2026-09-12 23:01:22 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Pass a triggering principal when launching non-HTTP URLs
Some checks failed
CI / Detect changes (push) Has been cancelled
Details
CI / Utilities Tests (push) Has been cancelled
Details
CI / Build, Upload (push) Has been cancelled
Details
CI / Test (macOS NFS) (push) Has been cancelled
Details
CI / Test (Windows x64) (push) Has been cancelled
Details
CI / Test () (push) Has been cancelled
Details
CI / Test (Windows arm64) (push) Has been cancelled
Details

Browse source 
Gecko 140.15 made nsIExternalProtocolService.loadURI()'s triggering
principal mandatory, so Zotero.launchURL() threw NS_ERROR_ILLEGAL_VALUE
for any scheme handled by an external app.

https://forums.zotero.org/discussion/133709/
This commit is contained in:
Dan Stillman 2026-09-11 14:31:41 -04:00
parent ab34e9031a
commit 3c959e1012
1 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
chrome/content/zotero/xpcom/zotero.js
View file

@ -1257,7 +1257,10 @@ const { CommandLineOptions } = ChromeUtils.importESModule("chrome://zotero/conte
Zotero.Utilities.Internal.Environment.clearMozillaVariables();
}
svc.loadURI(Services.io.newURI(url, null, null));
svc.loadURI(
Services.io.newURI(url, null, null),
Services.scriptSecurityManager.getSystemPrincipal(),
);
return;
}
}