Skip to content

Commit 79ddedc

Browse files
authored
Remove per second email throttling (#548)
1 parent c0c8fcd commit 79ddedc

File tree

2 files changed

+0
-26
lines changed

2 files changed

+0
-26
lines changed

backend/app/Jobs/Event/SendEventEmailJob.php

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -5,15 +5,13 @@
55
use HiEvents\DomainObjects\Generated\OutgoingMessageDomainObjectAbstract;
66
use HiEvents\DomainObjects\Status\OutgoingMessageStatus;
77
use HiEvents\Mail\Event\EventMessage;
8-
use HiEvents\Providers\AppServiceProvider;
98
use HiEvents\Repository\Interfaces\OutgoingMessageRepositoryInterface;
109
use HiEvents\Services\Application\Handlers\Message\DTO\SendMessageDTO;
1110
use Illuminate\Bus\Queueable;
1211
use Illuminate\Contracts\Queue\ShouldQueue;
1312
use Illuminate\Foundation\Bus\Dispatchable;
1413
use Illuminate\Mail\Mailer;
1514
use Illuminate\Queue\InteractsWithQueue;
16-
use Illuminate\Queue\Middleware\RateLimited;
1715
use Illuminate\Queue\SerializesModels;
1816
use Throwable;
1917

@@ -30,13 +28,6 @@ public function __construct(
3028
{
3129
}
3230

33-
public function middleware(): array
34-
{
35-
return [
36-
(new RateLimited(AppServiceProvider::MAIL_RATE_LIMIT_PER_SECOND)),
37-
];
38-
}
39-
4031
/**
4132
* @throws Throwable
4233
*/

backend/app/Providers/AppServiceProvider.php

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -12,21 +12,16 @@
1212
use HiEvents\Services\Infrastructure\CurrencyConversion\CurrencyConversionClientInterface;
1313
use HiEvents\Services\Infrastructure\CurrencyConversion\NoOpCurrencyConversionClient;
1414
use HiEvents\Services\Infrastructure\CurrencyConversion\OpenExchangeRatesCurrencyConversionClient;
15-
use Illuminate\Cache\RateLimiting\Limit;
16-
use Illuminate\Contracts\Queue\ShouldQueue;
1715
use Illuminate\Database\Eloquent\Model;
1816
use Illuminate\Database\Eloquent\Relations\Relation;
1917
use Illuminate\Support\Facades\DB;
2018
use Illuminate\Support\Facades\File;
21-
use Illuminate\Support\Facades\RateLimiter;
2219
use Illuminate\Support\Facades\URL;
2320
use Illuminate\Support\ServiceProvider;
2421
use Stripe\StripeClient;
2522

2623
class AppServiceProvider extends ServiceProvider
2724
{
28-
public const MAIL_RATE_LIMIT_PER_SECOND = 'mail-rate-limit-per-second';
29-
3025
public function register(): void
3126
{
3227
$this->bindDoctrineConnection();
@@ -46,18 +41,6 @@ public function boot(): void
4641
$this->disableLazyLoading();
4742

4843
$this->registerMorphMaps();
49-
50-
$this->registerJobRateLimiters();
51-
}
52-
53-
private function registerJobRateLimiters(): void
54-
{
55-
RateLimiter::for(
56-
name: self::MAIL_RATE_LIMIT_PER_SECOND,
57-
callback: static fn(ShouldQueue $job) => Limit::perMinute(
58-
maxAttempts: config('mail.rate_limit_per_second')
59-
)
60-
);
6144
}
6245

6346
private function bindDoctrineConnection(): void

0 commit comments

Comments
 (0)