feat(services): add log and login/session services#982
Open
karlitschek wants to merge 4 commits into
Open
Conversation
ChristophWurst
added a commit
that referenced
this pull request
May 11, 2026
Injects LogTailReader, LoginStats, ActivityRate, and ActiveConnections — added in #982 — into AdminSettings and passes their data to the template params, making the services active instead of dead code. Extracted from #977. Co-Authored-By: Frank Karlitschek <karlitschek@users.noreply.github.com> AI-assisted: Claude Code (claude-sonnet-4-6) Signed-off-by: Christoph Wurst <1374172+ChristophWurst@users.noreply.github.com>
ChristophWurst
force-pushed
the
split/05-services-logs-sessions
branch
from
7684654 to
b066758
Compare
Adds four read-only service classes used by the upcoming admin
dashboard cards. None are wired into existing controllers yet, so
this change is a pure addition with no behavior change.
* LogTailReader - last N WARN/ERROR entries from the JSON log
(no shell, line-bounded)
* LoginStats - bruteforce attempt counters and the top
offending IPs over recent windows
* ActivityRate - oc_activity row counts over the last hour /
day / week
* ActiveConnections - session counts derived from oc_authtoken
Signed-off-by: Frank Karlitschek <karlitschek@users.noreply.github.com>
ChristophWurst
added a commit
that referenced
this pull request
May 11, 2026
Injects LogTailReader, LoginStats, ActivityRate, and ActiveConnections — added in #982 — into AdminSettings and passes their data to the template params, making the services active instead of dead code. Extracted from #977. Co-Authored-By: Frank Karlitschek <karlitschek@users.noreply.github.com> AI-assisted: Claude Code (claude-sonnet-4-6) Signed-off-by: Christoph Wurst <1374172+ChristophWurst@users.noreply.github.com>
ChristophWurst
force-pushed
the
split/05-services-logs-sessions
branch
from
b066758 to
b34e5c4
Compare
ChristophWurst
added a commit
that referenced
this pull request
May 11, 2026
Injects LogTailReader, LoginStats, ActivityRate, and ActiveConnections — added in #982 — into AdminSettings and passes their data to the template params, making the services active instead of dead code. Extracted from #977. Co-Authored-By: Frank Karlitschek <karlitschek@users.noreply.github.com> AI-assisted: Claude Code (claude-sonnet-4-6) Signed-off-by: Christoph Wurst <1374172+ChristophWurst@users.noreply.github.com>
ChristophWurst
force-pushed
the
split/05-services-logs-sessions
branch
from
b34e5c4 to
8c232c6
Compare
ChristophWurst
added a commit
that referenced
this pull request
May 11, 2026
Injects LogTailReader, LoginStats, ActivityRate, and ActiveConnections — added in #982 — into AdminSettings and passes their data to the template params, making the services active instead of dead code. Extracted from #977. Co-Authored-By: Frank Karlitschek <karlitschek@users.noreply.github.com> AI-assisted: Claude Code (claude-sonnet-4-6) Signed-off-by: Christoph Wurst <1374172+ChristophWurst@users.noreply.github.com>
ChristophWurst
force-pushed
the
split/05-services-logs-sessions
branch
from
8c232c6 to
9a7a730
Compare
ChristophWurst
added a commit
that referenced
this pull request
May 12, 2026
Injects LogTailReader, LoginStats, ActivityRate, and ActiveConnections — added in #982 — into AdminSettings and passes their data to the template params, making the services active instead of dead code. Extracted from #977. Co-Authored-By: Frank Karlitschek <karlitschek@users.noreply.github.com> AI-assisted: Claude Code (claude-sonnet-4-6) Signed-off-by: Christoph Wurst <1374172+ChristophWurst@users.noreply.github.com>
ChristophWurst
force-pushed
the
split/05-services-logs-sessions
branch
from
9a7a730 to
28626fa
Compare
| $qb = $this->db->getQueryBuilder(); | ||
| $qb->select('subjectparams', 'type') | ||
| ->selectAlias($qb->func()->count('activity_id'), 'count') | ||
| ->from('activity') |
Member
There was a problem hiding this comment.
Member
|
Activity, auth token and rate limiting are conceptually problematic. This creates an inter app dependency where this app accesses the persistence of another app directly. The technically sound way would be to go through a public API. Looking into these right now … |
Member
|
Amended the original commit to fix the DCO and added class usage like it will be used later to avoid dead code. |
Member
ChristophWurst
left a comment
ChristophWurst
left a comment
There was a problem hiding this comment.
👍 functionality-wise
Needs decision conceptually
kesselb
reviewed
May 12, 2026
kesselb
reviewed
May 12, 2026
kesselb
reviewed
May 12, 2026
| 'cron' => $this->cronInfo->getCronInfo(), | ||
| 'jobQueue' => $this->jobQueueInfo->getJobQueueInfo(), | ||
| 'slowestJobs' => $this->slowestJobs->getSlowestJobs(), | ||
| 'logTail' => $this->logTailReader->recentErrors(), |
Collaborator
There was a problem hiding this comment.
Could we not bind them in getFrom but introduce a new controller (maybe ocs)?
This has the advantage to not block the main operation with slow operations (like expensive database queries, opening log files, etc) and also give us the ability to let the dashboard refresh without a page load in the long run.
Also okay as follow up, I would just mention it.
ChristophWurst
added a commit
that referenced
this pull request
May 12, 2026
Injects LogTailReader, LoginStats, ActivityRate, and ActiveConnections — added in #982 — into AdminSettings and passes their data to the template params, making the services active instead of dead code. Extracted from #977. Co-Authored-By: Frank Karlitschek <karlitschek@users.noreply.github.com> AI-assisted: Claude Code (claude-sonnet-4-6) Signed-off-by: Christoph Wurst <1374172+ChristophWurst@users.noreply.github.com>
ChristophWurst
force-pushed
the
split/05-services-logs-sessions
branch
from
28626fa to
7fbf7cc
Compare
Injects LogTailReader, LoginStats, ActivityRate, and ActiveConnections — added in #982 — into AdminSettings and passes their data to the template params, making the services active instead of dead code. Extracted from #977. Co-Authored-By: Frank Karlitschek <karlitschek@users.noreply.github.com> AI-assisted: Claude Code (claude-sonnet-4-6) Signed-off-by: Christoph Wurst <1374172+ChristophWurst@users.noreply.github.com>
ChristophWurst
force-pushed
the
split/05-services-logs-sessions
branch
from
e7fafb7 to
b48f36c
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds four read-only service classes used by the upcoming admin dashboard cards. None are wired into existing controllers yet, so this change is a pure addition with no behavior change.
(no shell, line-bounded)
offending IPs over recent windows
day / week