Incorrect detection of Firefox in Linux

[speckly]

Was wondering why I could not find the cookie for a specific domain despite it being stored. I found out the issue was that self.cookie_file was the path to my Firefox Developer Edition so I changed the path to the correct Firefox and it worked.

class FirefoxBased:
    """Superclass for Firefox based browsers"""

    def __init__(self, browser_name, cookie_file=None, domain_name="", **kwargs):
        self.browser_name = browser_name
        self.cookie_file = cookie_file or self.__find_cookie_file(**kwargs)
        self.cookie_file = "/home/speckles/.mozilla/firefox/g5cxou3r.default-release-1715532506105/cookies.sqlite" # TODO: Remove this!!!!!!

Perhaps its something to do with __expand_and_check_path in the same class as it returns the first result? edit: yeah it does seem like it imo

i use arch btw

[Atemu]

Experiencing the same on NixOS.

I have a bunch of profiles laying around but they're not the default. browser_cookie3 should use the default profile as selected in Firefox.

[kqvanity]

I had to add my desired profile manually at `` line ~913

elif sys.platform.startswith('linux') or 'bsd' in sys.platform.lower():
    user_data_path = self.__expand_and_check_path(linux_data_dirs)
    user_data_path += '/voyqppej.default-release-3'
elif sys.platform == 'win32':

Maybe the containing function should allow optional profile name (much like the domain)

[ErikDB87]

I experienced the same issue, and I was also able to fix it by manually defining self.cookie_file. So I assume there must be a bug in function __expand_and_check_path.

[raxod502]

Would a pull request be welcome to change the default behavior to be to load whatever profile is configured as the default in Firefox, and to introduce an option to select a specific profile if needed?

Edit: this is fixed by #126, which has not been merged yet.

If you wish to use the fix in your own code without needing to fork the dependency, here is a monkeypatching snippet:

Details

def monkeypatch_browser_cookie3():
    """
    Fix https://github.com/borisbabic/browser_cookie3/issues/211
    """

    import browser_cookie3

    import configparser
    import glob
    import os

    def get_default_profile(user_data_path):
        config = configparser.ConfigParser()
        profiles_ini_path = glob.glob(
            os.path.join(user_data_path + "**", "profiles.ini")
        )
        fallback_path = user_data_path + "**"

        if not profiles_ini_path:
            return fallback_path

        profiles_ini_path = profiles_ini_path[0]
        config.read(profiles_ini_path, encoding="utf8")

        profile_path = None
        for section in config.sections():
            if section.startswith("Install"):
                # Use that last Install section
                profile_path = config[section].get("Default")
            # in ff 72.0.1, if both an Install section and one with Default=1 are present, the former takes precedence
            elif config[section].get("Default") == "1" and not profile_path:
                profile_path = config[section].get("Path")

        for section in config.sections():
            # the Install section has no relative/absolute info, so check the profiles
            if config[section].get("Path") == profile_path:
                absolute = config[section].get("IsRelative") == "0"
                return (
                    profile_path
                    if absolute
                    else os.path.join(os.path.dirname(profiles_ini_path), profile_path)
                )

        return fallback_path

    browser_cookie3.FirefoxBased.get_default_profile = get_default_profile