slevomat / csob-gateway

Client library for CSOB card payment gateway
MIT License
27 stars 10 forks source link

Logging API request with PSR-3 #15

Closed x3wil closed 8 years ago

pepakriz commented 8 years ago

Jenom na zamyšlení. Co kdyby se to udělalo více obecně, nezávislé na PSR-3? V podstatě by si tu privátní metodu logRequest mohl každý naimplementovat po svém.

$myLogger = .....;
$apiClient->registerRequestLogger(function (string $url, array $queries, array $requestData = null, Response $response) use ($myLogger) {
    ...
    $myLogger->info($url, $context);
});
janlanger commented 8 years ago

@pepakriz to se mi nelíbí, je to "jen callback" jehož signatura se nedá vynutit. PSR-3 jako de facto standard mi přijde jednodušší na použití...

janlanger commented 8 years ago

@3wil ještě přidej psr/log do suggest v composer.json

pepakriz commented 8 years ago

@janlanger Ok

pepakriz commented 8 years ago

@3wil Nechceš zkusit na to napsat test? ;)