aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorpukkandan <pukkandan.ytdlp@gmail.com>2023-06-19 14:45:59 +0530
committerpukkandan <pukkandan.ytdlp@gmail.com>2023-06-21 06:10:38 +0530
commit2e023649ea4e11151545a34dc1360c114981a236 (patch)
tree821c0ba67fc040ab8f46a5181ed4be7abfb278a3
parent51a07b0dca4c079d58311c19b6d1c097c24bb021 (diff)
downloadhypervideo-pre-2e023649ea4e11151545a34dc1360c114981a236.tar.lz
hypervideo-pre-2e023649ea4e11151545a34dc1360c114981a236.tar.xz
hypervideo-pre-2e023649ea4e11151545a34dc1360c114981a236.zip
[cookies] Revert compatibility breakage in b38d4c941d1993ab27e4c0f8e024e23c2ec0f8f8
-rw-r--r--README.md2
-rw-r--r--yt_dlp/cookies.py20
2 files changed, 11 insertions, 11 deletions
diff --git a/README.md b/README.md
index 659730410..c39e151d4 100644
--- a/README.md
+++ b/README.md
@@ -728,7 +728,7 @@ If you fork the project on GitHub, you can run your fork's [build workflow](.git
By default, all containers of the most
recently accessed profile are used.
Currently supported keyrings are: basictext,
- gnomekeyring, kwallet
+ gnomekeyring, kwallet, kwallet5, kwallet6
--no-cookies-from-browser Do not load cookies from browser (default)
--cache-dir DIR Location in the filesystem where yt-dlp can
store some downloaded information (such as
diff --git a/yt_dlp/cookies.py b/yt_dlp/cookies.py
index e46d19341..8693e0b4a 100644
--- a/yt_dlp/cookies.py
+++ b/yt_dlp/cookies.py
@@ -705,11 +705,11 @@ class _LinuxKeyring(Enum):
https://chromium.googlesource.com/chromium/src/+/refs/heads/main/components/os_crypt/sync/key_storage_util_linux.h
SelectedLinuxBackend
"""
- KWALLET4 = auto() # this value is just called KWALLET in the chromium source but it is for KDE4 only
+ KWALLET = auto() # KDE4
KWALLET5 = auto()
KWALLET6 = auto()
- GNOME_KEYRING = auto()
- BASIC_TEXT = auto()
+ GNOMEKEYRING = auto()
+ BASICTEXT = auto()
SUPPORTED_KEYRINGS = _LinuxKeyring.__members__.keys()
@@ -803,7 +803,7 @@ def _choose_linux_keyring(logger):
desktop_environment = _get_linux_desktop_environment(os.environ, logger)
logger.debug(f'detected desktop environment: {desktop_environment.name}')
if desktop_environment == _LinuxDesktopEnvironment.KDE4:
- linux_keyring = _LinuxKeyring.KWALLET4
+ linux_keyring = _LinuxKeyring.KWALLET
elif desktop_environment == _LinuxDesktopEnvironment.KDE5:
linux_keyring = _LinuxKeyring.KWALLET5
elif desktop_environment == _LinuxDesktopEnvironment.KDE6:
@@ -811,9 +811,9 @@ def _choose_linux_keyring(logger):
elif desktop_environment in (
_LinuxDesktopEnvironment.KDE3, _LinuxDesktopEnvironment.LXQT, _LinuxDesktopEnvironment.OTHER
):
- linux_keyring = _LinuxKeyring.BASIC_TEXT
+ linux_keyring = _LinuxKeyring.BASICTEXT
else:
- linux_keyring = _LinuxKeyring.GNOME_KEYRING
+ linux_keyring = _LinuxKeyring.GNOMEKEYRING
return linux_keyring
@@ -828,7 +828,7 @@ def _get_kwallet_network_wallet(keyring, logger):
"""
default_wallet = 'kdewallet'
try:
- if keyring == _LinuxKeyring.KWALLET4:
+ if keyring == _LinuxKeyring.KWALLET:
service_name = 'org.kde.kwalletd'
wallet_path = '/modules/kwalletd'
elif keyring == _LinuxKeyring.KWALLET5:
@@ -929,11 +929,11 @@ def _get_linux_keyring_password(browser_keyring_name, keyring, logger):
keyring = _LinuxKeyring[keyring] if keyring else _choose_linux_keyring(logger)
logger.debug(f'Chosen keyring: {keyring.name}')
- if keyring in (_LinuxKeyring.KWALLET4, _LinuxKeyring.KWALLET5, _LinuxKeyring.KWALLET6):
+ if keyring in (_LinuxKeyring.KWALLET, _LinuxKeyring.KWALLET5, _LinuxKeyring.KWALLET6):
return _get_kwallet_password(browser_keyring_name, keyring, logger)
- elif keyring == _LinuxKeyring.GNOME_KEYRING:
+ elif keyring == _LinuxKeyring.GNOMEKEYRING:
return _get_gnome_keyring_password(browser_keyring_name, logger)
- elif keyring == _LinuxKeyring.BASIC_TEXT:
+ elif keyring == _LinuxKeyring.BASICTEXT:
# when basic text is chosen, all cookies are stored as v10 (so no keyring password is required)
return None
assert False, f'Unknown keyring {keyring}'