Uploaded image for project: 'ListenBrainz'
  1. ListenBrainz
  2. LB-926

Handle rate limiting headers from API in the frontend

    • Icon: Improvement Improvement
    • Resolution: Unresolved
    • Icon: Normal Normal
    • None
    • None
    • None

      We have a number of errors in sentry from the frontend which receive an HTTP429 error from the API (too many requests) - search sentry for "You have exceeded your rate limit ". This seems to mostly happen in the importer, but also in the listen view and reports page.

      We should update the API calls in the frontend to take care of this. Now that we have a single APIService it should be easy to centralise this.

      There is already a small amount of code in the APIServcice that seems to try and handle this status, but it's not used in all requests.

          Loading...
          Uploaded image for project: 'ListenBrainz'
          1. ListenBrainz
          2. LB-926

          Handle rate limiting headers from API in the frontend

            • Icon: Improvement Improvement
            • Resolution: Unresolved
            • Icon: Normal Normal
            • None
            • None
            • None

              We have a number of errors in sentry from the frontend which receive an HTTP429 error from the API (too many requests) - search sentry for "You have exceeded your rate limit ". This seems to mostly happen in the importer, but also in the listen view and reports page.

              We should update the API calls in the frontend to take care of this. Now that we have a single APIService it should be easy to centralise this.

              There is already a small amount of code in the APIServcice that seems to try and handle this status, but it's not used in all requests.

                    Unassigned Unassigned
                    alastairp Alastair Porter
                    Votes:
                    0 Vote for this issue
                    Watchers:
                    2 Start watching this issue

                      Created:
                      Updated:

                        Version Package

                          Unassigned Unassigned
                          alastairp Alastair Porter
                          Votes:
                          0 Vote for this issue
                          Watchers:
                          2 Start watching this issue

                            Created:
                            Updated:

                              Version Package