Page MenuHomePhorge

15 messages / minute rate-limit for paying users
Needs RevisionPublic

Authored by mollekopf on Nov 28 2023, 7:54 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Sep 8, 5:16 PM
Unknown Object (File)
Sat, Sep 7, 6:14 PM
Unknown Object (File)
Tue, Sep 3, 9:30 PM
Unknown Object (File)
Tue, Sep 3, 6:28 AM
Unknown Object (File)
Sun, Sep 1, 3:08 AM
Unknown Object (File)
Thu, Aug 29, 2:42 AM
Unknown Object (File)
Mon, Aug 26, 12:10 AM
Unknown Object (File)
Tue, Aug 20, 5:11 AM
Subscribers

Details

Reviewers
machniak
Group Reviewers
Restricted Project

Diff Detail

Repository
rK kolab
Branch
dev/mollekopf
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 45880
Build 17755: arc lint + arc unit

Event Timeline

mollekopf created this revision.
mollekopf added a reviewer: Restricted Project.Nov 28 2023, 8:03 AM
machniak added inline comments.
src/app/Http/Controllers/API/V4/PolicyController.php
148

This will make the 100%-discounted users also being limited. I'm not sure it's intentional.

149

This way the limit is shared by all users in an account. Might be too small for bigger accounts. How about where('user_id', $user->id) instead?

151

No need for $count variable.

src/app/Wallet.php
754

The argument is not documented. And maybe it should be $minPaymentsCount.

This revision now requires changes to proceed.May 23 2024, 10:43 AM