Retry listenbrainz requests for temporary failures

Author: aereaux

beetsplug/listenbrainz.py

@@ -15,6 +15,7 @@
 
 from ._utils.musicbrainz import MusicBrainzAPIMixin
 from ._utils.playcount import update_play_counts
+from ._utils.requests import TimeoutAndRetrySession
 
 if TYPE_CHECKING:
     from ._utils.playcount import Track
@@ -34,6 +35,7 @@ def __init__(self):
         super().__init__()
         self.token = self.config["token"].get()
         self.username = self.config["username"].get()
+        self.session = TimeoutAndRetrySession()
         self.AUTH_HEADER = {"Authorization": f"Token {self.token}"}
         config["listenbrainz"]["token"].redact = True
 
@@ -108,7 +110,7 @@ def _make_request(self, url, params=None):
         returning, so the next call is guaranteed a fresh quota.
         """
         try:
-            response = requests.get(
+            response = self.session.get(
                 url=url,
                 headers=self.AUTH_HEADER,
                 timeout=10,

docs/changelog.rst

@@ -26,9 +26,11 @@ New features
       after upgrading to trigger the migration. Only then you can safely move
       the library to a new location.
 
-..
-    Bug fixes
-    ~~~~~~~~~
+Bug fixes
+~~~~~~~~~
+
+- :doc:`plugins/listenbrainz`: Retry listenbrainz requests for temporary
+  failures.
 
 ..
     For plugin developers