Closed
Bug 1404956
Opened 7 years ago
Closed 7 years ago
Regression: 1Password no longer recognizes Firefox Nightly due to changed bundle identifier
Categories
(Firefox :: Untriaged, defect)
Tracking
()
VERIFIED
FIXED
Tracking | Status | |
---|---|---|
firefox56 | --- | unaffected |
firefox57 | --- | unaffected |
firefox58 | --- | verified |
People
(Reporter: jamie, Unassigned)
References
Details
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_6) AppleWebKit/604.1.38 (KHTML, like Gecko) Version/11.0 Safari/604.1.38
Steps to reproduce:
* Install Firefox Nightly 58
* Install 1Password and the 1Password add-on in Firefox Nightly 58
Notes:
This is similar to Bug 1404823, but it affects Firefox Nightly's identity on the system. Firefox's bundle identifier in Info.plist is set based on MOZ_APP_DISPLAYNAME, which is changed as part of Bug 1378834 (See first patch). Instead, I would propose basing the bundle identifier off BrandFullNameInternal instead, but perhaps there is a better way.
Actual results:
1Password no longer recognizes Firefox 58
Expected results:
1Password recognizes Firefox 58
Comment 1•7 years ago
|
||
Hey Jamie, sounds like bug 1404796 addresses this, can you confirm?
Reporter | ||
Comment 2•7 years ago
|
||
(In reply to Kev Needham [:kev] from comment #1)
> Hey Jamie, sounds like bug 1404796 addresses this, can you confirm?
I don't think so, Kevin. Previously, the bundle identifier was org.mozilla.nightly. Now, with the change to MOZ_APP_DISPLAY_NAME, it is org.mozilla.firefoxnightly.
And actually, the patch in bug 1404796 will only make things worse. The code that sets the bundle identifier simply uses `tr` to downcase the MOZ_APP_DISPLAY_NAME value:
MOZ_MACBUNDLE_ID=`echo $MOZ_APP_DISPLAYNAME | tr '[A-Z]' '[a-z]'`
Adding the space will make the bundle identifier entirely invalid because spaces are not allowed: https://developer.apple.com/library/content/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html#//apple_ref/doc/uid/TP40009249-102070-TPXREF105
Comment 3•7 years ago
|
||
> Adding the space will make the bundle identifier entirely invalid because spaces are not allowed
For this, we could also remove the space in the MOZ_MACBUNDLE_ID variable.
We could also change the definition of MOZ_MACBUNDLE_ID to be org.mozilla.nightly as it was.
Reporter | ||
Comment 4•7 years ago
|
||
I think changing it back to org.mozilla.nightly is the path of least disruption.
Comment 6•7 years ago
|
||
Clearing n-i as this is being backed out for now.
Flags: needinfo?(spohl.mozilla.bugs)
Comment 7•7 years ago
|
||
This issue was fixed by the backed out changes from bug 1378834.
Status: UNCONFIRMED → RESOLVED
Closed: 7 years ago
status-firefox56:
--- → unaffected
status-firefox57:
--- → unaffected
status-firefox58:
--- → fixed
Depends on: 1378834
OS: Unspecified → Mac OS X
Hardware: Unspecified → All
Resolution: --- → FIXED
Updated•7 years ago
|
Comment 8•7 years ago
|
||
osascript -e 'id of app "Firefox Nightly"'
org.mozilla.nightly
Also, 1Password seems now to properly identify Firefox Nightly correctly. Not sure if this suffices in order to mark this issue as verified, or if there is anything that should be additionally checked.
Jamie, Sylvestre, thoughts on any specific tests related to the bundle identifier that should be ran before marking this issue as verified or areas of risk?
Flags: needinfo?(sledru)
Flags: needinfo?(jamie)
Reporter | ||
Comment 9•7 years ago
|
||
After the initial kerfuffle, things have calmed down on this front. Basically, 1Password will not allow any connection or offer any functionality if the bundle identifier is not an expected value. 1Password understands the following values for a few different Firefox variants:
* org.mozilla.firefox
* org.mozilla.aurora
* org.mozilla.firefoxdeveloperedition
* org.mozilla.nightly
The issue here was that the bundle id changed unexpectedly. Now that it's back to just .nightly, I think we're in good shape.
Flags: needinfo?(jamie)
Updated•7 years ago
|
Flags: needinfo?(sledru)
Comment 10•7 years ago
|
||
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•