Page MenuHomePhorge
Differential D4890

Exempt liveness checks from logging
ClosedPublic
Actions

Authored by mollekopf on Aug 23 2024, 12:01 PM.
Tags
None
Referenced Files
F16772304: D4890.diff
Sun, Nov 10, 7:09 AM
Unknown Object (File)
Sun, Oct 27, 2:43 PM
Unknown Object (File)
Mon, Oct 14, 2:46 AM
Unknown Object (File)
Oct 11 2024, 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
View All 41 Files
Subscribers
machniak

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 Not Applicable
Unit
Tests Not Applicable

Event Timeline

mollekopf requested review of this revision.Aug 23 2024, 12:01 PM
mollekopf created this revision.
Harbormaster completed remote builds in B48795: Diff 14028.Aug 23 2024, 12:01 PM
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.
mollekopf added a comment.Aug 23 2024, 12:07 PM

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 requested changes to this revision.Aug 28 2024, 1:14 PM
machniak subscribed.
machniak added inline comments.
src/app/Http/Middleware/RequestLogger.php
23–27

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
Closed by commit rKc6410b01d297: Excempt liveness checks from logging (authored by mollekopf). · Explain Why
This revision was automatically updated to reflect the committed changes.
mollekopf added a commit: rKc6410b01d297: Excempt liveness checks from logging.

Revision Contents
Changeset List

PathSize
src/
app/
Http/
Controllers/
API/
V4/
8 lines
Middleware/
12 lines

Diff 14127

View Options

src/app/Http/Controllers/API/V4/HealthController.php

Loading...
View Options

src/app/Http/Middleware/RequestLogger.php

Loading...