Open Bug 1898283 Opened 1 year ago Updated 3 months ago

Use full profile path to compute DBus name

Categories

(Core :: Widget: Gtk, defect)

defect

Tracking

()

People

(Reporter: gerard-majax, Assigned: stransky, NeedInfo)

References

(Blocks 2 open bugs)

Details

Attachments

(2 files)

If one has multiple installations using the same profile name (but at difference places) e.g., like a packaged firefox (or snap) + tarball, this will clash on DBus names (broke me in bug 1897907)

Does Snap sandbox allow to get profile directory? I mean without sandbox we may be able to get profile name like /home/user/.mozilla/profile but but which profile patch we can get for sandboxed apps?

(In reply to Martin Stránský [:stransky] (ni? me) from comment #1)

Does Snap sandbox allow to get profile directory? I mean without sandbox we may be able to get profile name like /home/user/.mozilla/profile but but which profile patch we can get for sandboxed apps?

my current WIP just adds the profile path to compute the dbus name, I'm not changing anything else

ok you were too fast, i was also changing some code around nsDbusRemote* so we dont duplicate client/server side of the identifier computation

my nightly running from tarball with default profile used, and a Snap install started with a default profile as well

Assignee: lissyx+mozillians → stransky
Flags: needinfo?(stransky)
Attachment #9403447 - Attachment description: Bug 1898283 [Linux] Use full profile patch to identify remote application instance r?emilio → Bug 1898283 [Linux] Use profile directory to identify remote application instance r?emilio
Flags: needinfo?(stransky)
Flags: needinfo?(stransky)
See Also: → 1874870

There is an r+ patch which didn't land and no activity in this bug for 2 weeks.
:stransky, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit BugBot documentation.

Flags: needinfo?(stransky)
Flags: needinfo?(nalexander)

I'm not the right person to work on this patch; clearing NI.

Flags: needinfo?(nalexander)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: