automatic-releases icon indicating copy to clipboard operation
automatic-releases copied to clipboard
verified publisher icon laminas

Add more logging

Open geerteltink opened this issue 5 years ago • 3 comments

Q A
New Feature yes

Description

This PR ads more logging.

geerteltink avatar Sep 12 '20 10:09 geerteltink

I wasn't sure about logging in the command only or the services. I guess both is better.

geerteltink avatar Sep 14 '20 14:09 geerteltink

When doing work through the API, it would also be great to output the response content when getting an error response from the API instead of only having the failed invariant saying "something went wrong" (or a similar generic message).

stof avatar Sep 13 '22 11:09 stof

That is an excellent idea: adding logging to the PSR-18 client should suffice, I suppose?

Ocramius avatar Sep 13 '22 11:09 Ocramius

Closing here: to be retried in a new PR, if needed, but the rebase work is equivalent to the dev work here :)

Ocramius avatar Nov 15 '22 09:11 Ocramius