Differential D5710

api/throttling: Increase rate limit for authenticated users
ClosedPublic
Actions

Authored by anlambert on May 7 2021, 12:19 PM.

Details

Reviewers

ardumont

Group Reviewers

Reviewers

Maniphest Tasks

T3312: web API rate limit: 10x more quota for authenticated users

Commits

rDWAPPS2f85b65ddfbc: api/throttling: Increase rate limit for authenticated users

Summary

When a Web API user is authenticated, the number of allowed requests
by throttle scope is now increased by a factor of ten.

Related to T3312

Diff Detail

Repository

rDWAPPS Web applications

Branch

throttling-authenticated-users

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 21365
Build 33186: Phabricator diff pipeline on jenkins	Jenkins console · Jenkins
Build 33185: arc lint + arc unit

Event Timeline

anlambert created this revision.May 7 2021, 12:19 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptMay 7 2021, 12:19 PM

Fix typo and naming.

Build is green

Patch application report for D5710 (id=20385)

Rebasing onto 43636f47d5...

Current branch diff-target is up to date.

Changes applied before test

commit 81f68a7abddfd2407c598c8fda67aec5630920f8
Author: Antoine Lambert <antoine.lambert@inria.fr>
Date:   Fri May 7 12:15:21 2021 +0200

    api/throttling: Increase rate limit for authenticated users
    
    When a Web API user is authenticated, the number of allowed requests
    by throttle scope is now increased by a factor of ten.
    
    Related to T3312

See https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/795/ for more details.

Harbormaster completed remote builds in B21364: Diff 20385.May 7 2021, 12:27 PM

Build is green

Patch application report for D5710 (id=20386)

Rebasing onto 43636f47d5...

Current branch diff-target is up to date.

Changes applied before test

commit 2f85b65ddfbcf3121f4f2a642066a8930c2e3b32
Author: Antoine Lambert <antoine.lambert@inria.fr>
Date:   Fri May 7 12:15:21 2021 +0200

    api/throttling: Increase rate limit for authenticated users
    
    When a Web API user is authenticated, the number of allowed requests
    by throttle scope is now increased by a factor of ten.
    
    Related to T3312

See https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/796/ for more details.

Harbormaster completed remote builds in B21365: Diff 20386.May 7 2021, 12:35 PM

anlambert requested review of this revision.May 7 2021, 12:35 PM

ardumont accepted this revision.May 7 2021, 1:45 PM

This revision is now accepted and ready to land.May 7 2021, 1:45 PM

Closed by commit rDWAPPS2f85b65ddfbc: api/throttling: Increase rate limit for authenticated users (authored by anlambert). · Explain WhyMay 7 2021, 2:16 PM

This revision was automatically updated to reflect the committed changes.

anlambert added a commit: rDWAPPS2f85b65ddfbc: api/throttling: Increase rate limit for authenticated users.