Page MenuHomePhorge

./artisan status:health: check every backend instead of just one
ClosedPublic

Authored by mollekopf on Mar 27 2023, 10:30 AM.
Tags
None
Referenced Files
F11587281: D4208.id.diff
Thu, Mar 28, 5:03 PM
F11581695: D4208.id12062.diff
Thu, Mar 28, 2:43 AM
Unknown Object (File)
Sat, Mar 16, 12:03 PM
Unknown Object (File)
Fri, Mar 1, 2:26 AM
Unknown Object (File)
Fri, Mar 1, 2:25 AM
Unknown Object (File)
Fri, Mar 1, 2:25 AM
Unknown Object (File)
Fri, Mar 1, 2:25 AM
Unknown Object (File)
Fri, Mar 1, 2:25 AM
Subscribers

Details

Diff Detail

Repository
rK kolab
Lint
Lint Warnings
SeverityLocationCodeMessage
Warningsrc/app/Console/Commands/Status/Health.php:153PHPCS.W.Generic.Files.LineLength.TooLongGeneric.Files.LineLength.TooLong
Unit
No Test Coverage
Build Status
Buildable 42185
Build 17018: arc lint + arc unit

Event Timeline

mollekopf created this revision.
mollekopf added a reviewer: Restricted Project.Mar 27 2023, 11:33 AM
machniak subscribed.

An improvement would be to print which backend failed. Also, we shouldn't var_export the whole response object, error code/message would be enough.

This revision is now accepted and ready to land.Mar 27 2023, 11:49 AM

Only print the status code and reason

This revision was landed with ongoing or failed builds.Mar 27 2023, 2:43 PM
This revision was automatically updated to reflect the committed changes.