Page MenuHomePhorge

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

Authored by mollekopf on Nov 28 2023, 7:54 AM.
Tags
None
Referenced Files
F12073527: D4594.diff
Wed, May 8, 5:37 AM
Unknown Object (File)
Sat, May 4, 10:49 PM
Unknown Object (File)
Sun, Apr 28, 5:02 PM
Unknown Object (File)
Wed, Apr 24, 6:08 AM
Unknown Object (File)
Wed, Apr 17, 11:05 AM
Unknown Object (File)
Wed, Apr 17, 11:05 AM
Unknown Object (File)
Wed, Apr 17, 11:05 AM
Unknown Object (File)
Wed, Apr 17, 11:05 AM
Subscribers

Details

Reviewers
None
Group Reviewers
Restricted Project

Diff Detail

Repository
rK kolab
Branch
dev/mollekopf
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 45877
Build 17752: 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–157

No need for $count variable.

src/app/Wallet.php
754

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