While I do not object to this change, we should test and see if this change is needed. I have the impression (but no hard data) that the creator code is not used by OSX and that only the bundle ID is important. And I do not think Apple maintains a registry any more.
While I do not object to this change, we should test and see if this change is needed. I have the impression (but no hard data) that the creator code is not used by OSX and that only the bundle ID is important.
Apple has deprecated the creator code, but still uses it in their implementation for backwards-compatibility reasons.
But here's how I can reproduce the Tor Browser / Firefox confusion on macOS 10.12.3:
Install Firefox and Tor Browser
touch empty.html
In Finder, Get Info on empty.html
Set "Open With" to Tor Browser
Make sure both Firefox and Tor Browser are not running
Open empty.html with the Finder
(Either Firefox or Tor Browser will open depending on their order in the LaunchServices database, which depends on installation and maybe previous view/open order, and then if Tor Browser opens, Tor Launcher fails to handle the open request, so the default browser will open instead, see #20105 (moved))
Make sure Firefox is running and Tor Browser is not running
Open empty.html with the Finder
(Firefox opens, even though we said to Open in Tor Browser)
Make sure Tor Browser is running and Firefox is not running
Open empty.html with the Finder
(Tor Browser opens)
Make sure Tor Browser and Firefox are both running
Open empty.html with the Finder
(Either Firefox or Tor Browser will open depending on their order in the LaunchServices database, which depends on installation and maybe previous view/open/launch order)
What I expect to happen is that Tor Browser opens every time, and Firefox or the default browser do not open.
And I do not think Apple maintains a registry any more.
Yes, this is true. But we should try not to conflict with Mozilla's potential future uses, if any.
Kathy and I modified your patch to use 'TORB' instead of 'MOZT' (it seems like a good idea to include Tor somehow). We also rebased it so it will apply against the ESR52 code. As I said in #21273 (moved), trying these changes with our upcoming ESR52-based alpha seems like a good idea.
I had Tor Browser selected as my default browser (links automatically open in it). When I updated to Tor Browser 7.0, it stopped being my default browser, because the creator code changed due to this patch (or possibly #21723 (moved)).
This is what we expected to happen, but I thought I'd make a note here in case anyone reports this issue. Once users set 7.0 as the default browser, it will stay as the default, and it won't get confused with Firefox.
Trac: Keywords: N/Adeleted, tbb-no-uplift added Description: Please see my branch fix-macos-bundle-signature on https://github.com/teor2345/tor-browser.git , which deals with the core issue of #17670 (moved), the MOZB CFBundleSignature change.