External/domain-watchdog
1
0
Fork 0
mirror of https://github.com/maelgangloff/domain-watchdog.git synced 2025-12-29 16:15:04 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: correct HTTP codes for Bad Request

Browse Source
This commit is contained in:
Maël Gangloff
2025-10-07 16:33:27 +02:00
parent ea1df9d97a
commit f2c52eded1
3 changed files with 12 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
config/packages/api_platform.yaml
View File

@@ -38,6 +38,6 @@ api_platform:
ApiPlatform\Validator\Exception\ValidationException: !php/const Symfony\Component\HttpFoundation\Response::HTTP_UNPROCESSABLE_ENTITY ApiPlatform\Validator\Exception\ValidationException: !php/const Symfony\Component\HttpFoundation\Response::HTTP_UNPROCESSABLE_ENTITY
App\Exception\DomainNotFoundException: 404 App\Exception\DomainNotFoundException: 404
App\Exception\MalformedDomainException: 403 App\Exception\MalformedDomainException: 400
App\Exception\TldNotSupportedException: 403 App\Exception\TldNotSupportedException: 400
App\Exception\UnknownRdapServerException: 403 App\Exception\UnknownRdapServerException: 400

3
src/MessageHandler/UpdateDomainsFromWatchlistHandler.php
View File

@@ -41,7 +41,8 @@ final readonly class UpdateDomainsFromWatchlistHandler
private WatchListRepository $watchListRepository, private WatchListRepository $watchListRepository,
private LoggerInterface $logger, private LoggerInterface $logger,
#[Autowire(service: 'service_container')] #[Autowire(service: 'service_container')]
private ContainerInterface $locator, private DomainRepository $domainRepository, private ContainerInterface $locator,
private DomainRepository $domainRepository,
) { ) {
$this->sender = new Address($mailerSenderEmail, $mailerSenderName); $this->sender = new Address($mailerSenderEmail, $mailerSenderName);
} }

7
src/Service/RDAPService.php
View File

@@ -224,6 +224,9 @@ class RDAPService
$tldEntity = $this->tldRepository->findOneBy(['tld' => $tld]); $tldEntity = $this->tldRepository->findOneBy(['tld' => $tld]);
if (null === $tldEntity) { if (null === $tldEntity) {
$this->logger->warning('Domain name cannot be updated because the TLD is not supported', [
'ldhName' => $domain,
]);
throw TldNotSupportedException::fromTld($domain); throw TldNotSupportedException::fromTld($domain);
} }
@@ -244,6 +247,10 @@ class RDAPService
$rdapServer = $this->rdapServerRepository->findOneBy(['tld' => $tldString], ['updatedAt' => 'DESC']); $rdapServer = $this->rdapServerRepository->findOneBy(['tld' => $tldString], ['updatedAt' => 'DESC']);
if (null === $rdapServer) { if (null === $rdapServer) {
$this->logger->warning('Unable to determine which RDAP server to contact', [
'tld' => $tldString,
]);
throw UnknownRdapServerException::fromTld($tldString); throw UnknownRdapServerException::fromTld($tldString);
} }