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, May 26, 7:36 PM
Unknown Object (File)
Fri, May 24, 2:52 AM
Unknown Object (File)
Mon, May 20, 5:07 PM
Unknown Object (File)
Mon, May 20, 4:26 AM
Unknown Object (File)
Mon, May 20, 3:30 AM
Unknown Object (File)
Sun, May 19, 7:19 PM
Unknown Object (File)
Wed, May 15, 5:23 AM
Unknown Object (File)
Fri, May 10, 5:31 PM
Subscribers

Details

Reviewers
machniak
Group Reviewers
Restricted Project

Diff Detail

Repository
rK kolab
Branch
dev/mollekopf
Lint
Lint Warnings
SeverityLocationCodeMessage
Warningsrc/app/Wallet.php:749PHPCS.W.Generic.Files.LineLength.TooLongGeneric.Files.LineLength.TooLong
Unit
No Test Coverage
Build Status
Buildable 45874
Build 17749: 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.

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