Support username and password from keyring

src/hatch/publish/auth.py

@@ -1,5 +1,7 @@
 from __future__ import annotations
 
+import keyring
+
 from hatch.utils.fs import Path
 
 
@@ -44,12 +46,6 @@ def __get_password(self) -> str:
         if password is not None:
             return password
 
-        import keyring
-
-        password = keyring.get_password(self._repo, self.username)
-        if password is not None:
-            return password
-
         if self._options['no_prompt']:
             self._app.abort('Missing required option: auth')
 
@@ -62,6 +58,7 @@ def __get_username(self) -> str:
             or self._repo_config.get('user')
             or self._read_pypirc()
             or self._read_previous_working_user_data()
+            or self._read_keyring()
         )
         if username is not None:
             return username
@@ -72,6 +69,14 @@ def __get_username(self) -> str:
         self.__username_was_read = True
         return self._app.prompt(f"Username for '{self._repo_config['url']}' [__token__]") or '__token__'
 
+    def _read_keyring(self) -> str | None:
+        creds = keyring.get_credential(self._repo, None)
+        if not creds:
+            return None
+        self.__password = creds.password
+        self.__password_was_read = True
+        return creds.username
+
     def _read_previous_working_user_data(self) -> str | None:
         if self._pwu_path.is_file():
             contents = self._pwu_path.read_text()

tests/utils/test_auth.py

@@ -1,7 +1,24 @@
+import pytest
+
+import hatch.publish.auth
 from hatch.publish.auth import AuthenticationCredentials
 from hatch.utils.fs import Path
 
 
+@pytest.fixture(autouse=True)
+def mock_keyring(monkeypatch):
+    class MockKeyring:
+        @staticmethod
+        def get_credential(*_):
+            class Credential:
+                username = 'gat'
+                password = 'guido'
+
+            return Credential()
+
+    monkeypatch.setattr(hatch.publish.auth, 'keyring', MockKeyring)
+
+
 def test_pypirc(fs):
     fs.create_file(
         Path.home() / '.pypirc',
@@ -42,3 +59,16 @@ def test_pypirc(fs):
     )
     assert credentials.username == 'guido'
     assert credentials.password == 'gat'
+
+
+def test_keyring_credentials():
+    credentials = AuthenticationCredentials(
+        app=None,
+        cache_dir=Path('/none'),
+        options={},
+        repo='arbitrary',
+        repo_config={'url': 'https://kaashandel.nl/'},
+    )
+
+    assert credentials.username == 'gat'
+    assert credentials.password == 'guido'