feat(provider): implement ping and probe on GitHub, GitLab, and Gitea
Add getName() and ping() to each provider and wire ProbeTrait. Make all classes final, add strict_types declarations, and replace generic RuntimeException with typed HTTP exceptions so probe() can classify auth failures and unreachable endpoints correctly. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -1,10 +1,13 @@
|
|||||||
<?php
|
<?php
|
||||||
|
|
||||||
|
declare(strict_types=1);
|
||||||
|
|
||||||
namespace App\Service;
|
namespace App\Service;
|
||||||
|
|
||||||
use IDCI\Bundle\GraphQLClientBundle\Client\GraphQLApiClient;
|
use IDCI\Bundle\GraphQLClientBundle\Client\GraphQLApiClient;
|
||||||
use IDCI\Bundle\GraphQLClientBundle\Client\GraphQLApiClientRegistryInterface;
|
use IDCI\Bundle\GraphQLClientBundle\Client\GraphQLApiClientRegistryInterface;
|
||||||
use Psr\Log\LoggerInterface;
|
use Psr\Log\LoggerInterface;
|
||||||
|
use Symfony\Component\HttpKernel\Exception\ServiceUnavailableHttpException;
|
||||||
use Symfony\Contracts\HttpClient\HttpClientInterface;
|
use Symfony\Contracts\HttpClient\HttpClientInterface;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@@ -12,8 +15,10 @@ use Symfony\Contracts\HttpClient\HttpClientInterface;
|
|||||||
*
|
*
|
||||||
* Required token scopes: read:user
|
* Required token scopes: read:user
|
||||||
*/
|
*/
|
||||||
class GitHubProvider implements ProviderInterface
|
final class GitHubProvider implements ProviderInterface
|
||||||
{
|
{
|
||||||
|
use ProbeTrait;
|
||||||
|
|
||||||
private const GRAPHQL_URL = 'https://api.github.com/graphql';
|
private const GRAPHQL_URL = 'https://api.github.com/graphql';
|
||||||
|
|
||||||
public function __construct(
|
public function __construct(
|
||||||
@@ -24,11 +29,23 @@ class GitHubProvider implements ProviderInterface
|
|||||||
private readonly LoggerInterface $logger,
|
private readonly LoggerInterface $logger,
|
||||||
) {}
|
) {}
|
||||||
|
|
||||||
|
public function getName(): string
|
||||||
|
{
|
||||||
|
return 'github';
|
||||||
|
}
|
||||||
|
|
||||||
public function isConfigured(): bool
|
public function isConfigured(): bool
|
||||||
{
|
{
|
||||||
return $this->username !== '' && $this->token !== '';
|
return $this->username !== '' && $this->token !== '';
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public function ping(): void
|
||||||
|
{
|
||||||
|
$this->client->request('GET', 'https://api.github.com/user', [
|
||||||
|
'headers' => ['Authorization' => "Bearer {$this->token}"],
|
||||||
|
])->getContent();
|
||||||
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @return array<string, int> date (Y-m-d) => contribution count
|
* @return array<string, int> date (Y-m-d) => contribution count
|
||||||
*/
|
*/
|
||||||
@@ -69,7 +86,7 @@ class GitHubProvider implements ProviderInterface
|
|||||||
$data = $response->toArray();
|
$data = $response->toArray();
|
||||||
|
|
||||||
if (isset($data['errors'])) {
|
if (isset($data['errors'])) {
|
||||||
throw new \RuntimeException('GitHub GraphQL error: ' . json_encode($data['errors']));
|
throw new ServiceUnavailableHttpException(null, 'GitHub GraphQL error: ' . json_encode($data['errors']));
|
||||||
}
|
}
|
||||||
|
|
||||||
$result = [];
|
$result = [];
|
||||||
|
|||||||
@@ -1,8 +1,11 @@
|
|||||||
<?php
|
<?php
|
||||||
|
|
||||||
|
declare(strict_types=1);
|
||||||
|
|
||||||
namespace App\Service;
|
namespace App\Service;
|
||||||
|
|
||||||
use Psr\Log\LoggerInterface;
|
use Psr\Log\LoggerInterface;
|
||||||
|
use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
|
||||||
use Symfony\Contracts\HttpClient\HttpClientInterface;
|
use Symfony\Contracts\HttpClient\HttpClientInterface;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@@ -11,8 +14,10 @@ use Symfony\Contracts\HttpClient\HttpClientInterface;
|
|||||||
* Required token scopes: read_user, read_api
|
* Required token scopes: read_user, read_api
|
||||||
* Works with both gitlab.com and self-hosted instances.
|
* Works with both gitlab.com and self-hosted instances.
|
||||||
*/
|
*/
|
||||||
class GitLabProvider implements ProviderInterface
|
final class GitLabProvider implements ProviderInterface
|
||||||
{
|
{
|
||||||
|
use ProbeTrait;
|
||||||
|
|
||||||
public function __construct(
|
public function __construct(
|
||||||
private readonly HttpClientInterface $client,
|
private readonly HttpClientInterface $client,
|
||||||
private readonly string $username,
|
private readonly string $username,
|
||||||
@@ -21,11 +26,25 @@ class GitLabProvider implements ProviderInterface
|
|||||||
private readonly string $baseUrl = '',
|
private readonly string $baseUrl = '',
|
||||||
) {}
|
) {}
|
||||||
|
|
||||||
|
public function getName(): string
|
||||||
|
{
|
||||||
|
return 'gitlab';
|
||||||
|
}
|
||||||
|
|
||||||
public function isConfigured(): bool
|
public function isConfigured(): bool
|
||||||
{
|
{
|
||||||
return $this->username !== '' && $this->token !== '';
|
return $this->username !== '' && $this->token !== '';
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public function ping(): void
|
||||||
|
{
|
||||||
|
$baseUrl = rtrim($this->baseUrl !== '' ? $this->baseUrl : 'https://gitlab.com', '/');
|
||||||
|
|
||||||
|
$this->client->request('GET', "$baseUrl/api/v4/user", [
|
||||||
|
'headers' => ['PRIVATE-TOKEN' => $this->token],
|
||||||
|
])->getContent();
|
||||||
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @return array<string, int> date (Y-m-d) => event count
|
* @return array<string, int> date (Y-m-d) => event count
|
||||||
*/
|
*/
|
||||||
@@ -35,7 +54,6 @@ class GitLabProvider implements ProviderInterface
|
|||||||
|
|
||||||
$this->logger->debug('GitLabProvider: fetching contributions', ['user' => $this->username, 'url' => $baseUrl]);
|
$this->logger->debug('GitLabProvider: fetching contributions', ['user' => $this->username, 'url' => $baseUrl]);
|
||||||
|
|
||||||
// Resolve numeric user ID from username
|
|
||||||
$userResponse = $this->client->request('GET', "$baseUrl/api/v4/users", [
|
$userResponse = $this->client->request('GET', "$baseUrl/api/v4/users", [
|
||||||
'headers' => ['PRIVATE-TOKEN' => $this->token],
|
'headers' => ['PRIVATE-TOKEN' => $this->token],
|
||||||
'query' => ['username' => $this->username],
|
'query' => ['username' => $this->username],
|
||||||
@@ -43,7 +61,7 @@ class GitLabProvider implements ProviderInterface
|
|||||||
|
|
||||||
$users = $userResponse->toArray();
|
$users = $userResponse->toArray();
|
||||||
if (empty($users)) {
|
if (empty($users)) {
|
||||||
throw new \RuntimeException("GitLab: user '{$this->username}' not found on $baseUrl");
|
throw new NotFoundHttpException("GitLab: user '{$this->username}' not found on $baseUrl");
|
||||||
}
|
}
|
||||||
$userId = $users[0]['id'];
|
$userId = $users[0]['id'];
|
||||||
|
|
||||||
|
|||||||
@@ -1,5 +1,7 @@
|
|||||||
<?php
|
<?php
|
||||||
|
|
||||||
|
declare(strict_types=1);
|
||||||
|
|
||||||
namespace App\Service;
|
namespace App\Service;
|
||||||
|
|
||||||
use Psr\Log\LoggerInterface;
|
use Psr\Log\LoggerInterface;
|
||||||
@@ -13,8 +15,10 @@ use Symfony\Contracts\HttpClient\HttpClientInterface;
|
|||||||
*
|
*
|
||||||
* Required token scopes: read:user
|
* Required token scopes: read:user
|
||||||
*/
|
*/
|
||||||
class GiteaProvider implements ProviderInterface
|
final class GiteaProvider implements ProviderInterface
|
||||||
{
|
{
|
||||||
|
use ProbeTrait;
|
||||||
|
|
||||||
public function __construct(
|
public function __construct(
|
||||||
private readonly HttpClientInterface $client,
|
private readonly HttpClientInterface $client,
|
||||||
private readonly string $username,
|
private readonly string $username,
|
||||||
@@ -23,11 +27,25 @@ class GiteaProvider implements ProviderInterface
|
|||||||
private readonly LoggerInterface $logger,
|
private readonly LoggerInterface $logger,
|
||||||
) {}
|
) {}
|
||||||
|
|
||||||
|
public function getName(): string
|
||||||
|
{
|
||||||
|
return 'gitea';
|
||||||
|
}
|
||||||
|
|
||||||
public function isConfigured(): bool
|
public function isConfigured(): bool
|
||||||
{
|
{
|
||||||
return $this->username !== '' && $this->token !== '' && $this->baseUrl !== '';
|
return $this->username !== '' && $this->token !== '' && $this->baseUrl !== '';
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public function ping(): void
|
||||||
|
{
|
||||||
|
$baseUrl = rtrim($this->baseUrl, '/');
|
||||||
|
|
||||||
|
$this->client->request('GET', "$baseUrl/api/v1/user", [
|
||||||
|
'headers' => ['Authorization' => "token {$this->token}"],
|
||||||
|
])->getContent();
|
||||||
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @return array<string, int> date (Y-m-d) => contribution count
|
* @return array<string, int> date (Y-m-d) => contribution count
|
||||||
*/
|
*/
|
||||||
|
|||||||
Reference in New Issue
Block a user