• 
      

    Added optional feature to use Djblet's WebAPIResourceRateLimitMixin to rate limit API calls

    Review Request #8868 — Created April 3, 2017 and discarded

    Information

    Review Board
    master

    Reviewers

    Review Board needs the option to allow rate limits for any API
    requests (see review request 8839 for more details).
    This is done by allowing the user to override Djblet's
    WebAPIResourceRateLimitMixin class.

    The test for the WebAPIResourceRateLimitMixin class can be found
    in review request 8839. However, one test has been added in
    webapi/tests/test_base.py

    Description From Last Updated

    This isn't immediately useful. You will want to add a field to the admin settings form that saves whether or …

    brenniebrennie

    This isn't immediately useful. You will want to add a field to the admin settings form that saves whether or …

    brenniebrennie
    brennie
    1. 
        
    2. reviewboard/webapi/base.py (Diff revision 1)
       
       
      Show all issues

      This isn't immediately useful.

      You will want to add a field to the admin settings form that saves whether or not we should ratelimit to siteconfig and return that value here.

    3. reviewboard/webapi/base.py (Diff revision 1)
       
       
      Show all issues

      This isn't immediately useful.

      You will want to add a field to the admin settings form that saves whether or not we should ratelimit to siteconfig and return that value here.

    4. 
        
    david
    Review request changed
    Status:
    Discarded
    Change Summary:

    Discarding in favor of https://reviews.reviewboard.org/r/9114/