ekinhbayar / gitamp

server.php

<?php declare(strict_types = 1);

use Aerys\Host;
use Amp\Redis\Client;
use Auryn\Injector;
use ekinhbayar\GitAmp\Github\Credentials;
use ekinhbayar\GitAmp\Storage\Counter;
use ekinhbayar\GitAmp\Storage\RedisCounter;
use ekinhbayar\GitAmp\Websocket\Handler;
use Monolog\Logger;
use Monolog\Handler\StreamHandler;
use Psr\Log\LoggerInterface;
use function Aerys\root;
use function Aerys\router;
use function Aerys\websocket;

//require_once __DIR__ . '/vendor/autoload.php';

$configuration = require_once __DIR__ . '/config.php';

$injector = new Injector;

$injector->alias(Counter::class, RedisCounter::class);

$injector->alias(Credentials::class, get_class($configuration['github']));

$injector->share($configuration['github']);

$injector->alias(LoggerInterface::class, Logger::class);
$injector->share(Logger::class);

$injector->delegate(Logger::class, function() use ($configuration) {
    $logger = new Logger('gitamp');

    $logger->pushHandler(new StreamHandler('php://stdout', $configuration['logLevel']));

    return $logger;
});

// @todo unuglify this

Comments for TODO tasks are often forgotten in the code; it might be better to use a dedicated issue tracker.

if (isset($configuration['ssl'])) {
    $origin = 'https://' . $configuration['hostname'];

    if ($configuration['ssl']['port'] !== 443) {

This code seems to be duplicated across your project.

        $origin .= ':' . $configuration['ssl']['port'];
    }
} else {
    $origin = 'http://' . $configuration['hostname'];

    if ($configuration['expose']['port'] !== 80) {

This code seems to be duplicated across your project.

        $origin .= ':' . $configuration['expose']['port'];
    }
}

$injector->define(Handler::class, [
    ':origin' => $origin,
]);

$injector->define(Client::class, [
    ':uri' => $configuration['redis']
]);

$websocket = $injector->make(Handler::class);

$router = router()->get("/ws", websocket($websocket));

The string literal /ws does not require double quotes, as per coding-style, please use single quotes.

if (isset($configuration['ssl'])) {
    (new Host())
        ->name($configuration['hostname'])
        ->expose($configuration['expose']['ip'], $configuration['expose']['port'])
        ->redirect('https://' . $configuration['hostname'])
    ;

    (new Host())
        ->name($configuration['hostname'])
        ->expose($configuration['ssl']['ip'], $configuration['ssl']['port'])
        ->encrypt($configuration['ssl']['certificate'], $configuration['ssl']['key'])
        ->use($router)
        ->use(root(__DIR__ . '/public'))

\Aerys\root(__DIR__ . '/public') is of type object<Aerys\Bootable>, but the function expects a callable.

    ;
} else {
    (new Host())
        ->name($configuration['hostname'])
        ->expose($configuration['expose']['ip'], $configuration['expose']['port'])
        ->use($router)
        ->use(root(__DIR__ . '/public'))

\Aerys\root(__DIR__ . '/public') is of type object<Aerys\Bootable>, but the function expects a callable.

    ;
}

$logger = $injector->make(LoggerInterface::class);

\Amp\onError(function(Throwable $e) use ($logger) {
    $logger->emergency('GitAmp blew up', ['exception' => $e]);
});