Skip to content

feat: expand the serverstatus API surface - #440

Open
NickJosevski wants to merge 3 commits into
mainfrom
nj/serverstatus-endpoints
Open

feat: expand the serverstatus API surface#440
NickJosevski wants to merge 3 commits into
mainfrom
nj/serverstatus-endpoints

Conversation

@NickJosevski

@NickJosevski NickJosevski commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Refs #47.

ServerStatusService exposed only Get. Adds newclient functions for every endpoint on the ServerStatus resource:

Function Endpoint
GetServerStatus /api/serverstatus
GetHealthStatus /api/serverstatus/health
GetTimezones /api/serverstatus/timezones
GetDocumentCounts /api/serverstatus/counts
GetSystemInfo /api/serverstatus/system-info
GetRecentLogs /api/serverstatus/logs{?skip,take,includeDetail}
GetSystemReport /api/serverstatus/system-report

Also adds the missing IsPotentialClone field on ServerStatus.

Notes:

  • GetSystemReport returns an io.ReadCloser — the endpoint serves a zip archive, not JSON. The caller closes it.
  • GetSystemInfo, GetRecentLogs and GetSystemReport require the AdministerSystem permission.
  • SystemInfo.Uptime is left as the server's time span string; FromTimeSpan lives in machinepolicies/machines, so parsing it here would mean importing one of those.
  • /api/serverstatus/extensions is not included; it 404s and is not advertised on the resource.

All response shapes verified against a live server. e2e tests cover all seven, and unit tests pin the LogsQuery expansion.

NickJosevski and others added 3 commits August 6, 2026 19:05
…erver

ActionTemplateSearch is the response shape of Search and carries no uri
tags, so the fields passed to Get never reached the server and every call
returned the default first page. Deprecate Get and add GetByQuery, which
takes the existing Query.

Fixes #437

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Adds GetServerStatus, GetHealthStatus, GetTimezones and GetDocumentCounts
as newclient functions, and the missing IsPotentialClone field on
ServerStatus. Shapes verified against a live server.

Refs #47

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Adds GetSystemInfo, GetRecentLogs and GetSystemReport. The report is a zip
archive, so it returns an io.ReadCloser. All three require the
AdministerSystem permission. Shapes verified against a live server.

Refs #47

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@NickJosevski
NickJosevski marked this pull request as ready for review August 7, 2026 05:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant