Page MenuHomePhorge

Exempt liveness checks from logging
ClosedPublic

Authored by mollekopf on Aug 23 2024, 12:01 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Oct 27, 2:43 PM
Unknown Object (File)
Mon, Oct 14, 2:46 AM
Unknown Object (File)
Fri, Oct 11, 7:39 AM
Unknown Object (File)
Oct 4 2024, 5:40 PM
Unknown Object (File)
Oct 2 2024, 2:16 PM
Unknown Object (File)
Sep 30 2024, 10:35 AM
Unknown Object (File)
Sep 28 2024, 8:27 AM
Unknown Object (File)
Sep 28 2024, 8:06 AM
Subscribers

Details

Reviewers
machniak
Group Reviewers
Restricted Project
Commits
rKc6410b01d297: Excempt liveness checks from logging
Summary

In containerized environments we want to run liveness checks every couple of seconds, but there isn't really a point to log all these access attempts.
This just avoids logging the liveness checks.

Diff Detail

Repository
rK kolab
Lint
Lint Skipped
Unit
No Test Coverage
Build Status
Buildable 48795
Build 18321: arc lint + arc unit

Event Timeline

mollekopf created this revision.
mollekopf retitled this revision from Attempt to exempt liveness checks from logging to Exempt liveness checks from logging.Aug 23 2024, 12:03 PM
mollekopf edited the summary of this revision. (Show Details)
mollekopf added a reviewer: Restricted Project.

This currently only applies when APP_DEBUG=true, but having a log line every couple of seconds when debugging is still annoying enough IMO.

machniak subscribed.
machniak added inline comments.
src/app/Http/Middleware/RequestLogger.php
23

This would be better to be empty() instead !isset()

This revision now requires changes to proceed.Aug 28 2024, 1:14 PM
This revision was not accepted when it landed; it landed in state Needs Revision.Sep 6 2024, 2:20 PM
This revision was automatically updated to reflect the committed changes.