Deploymentcenter-Anbindung auf SDK 2.8.2 nachgezogen (#218)
Der Changelog seit 2.6.0 nennt sechs Fassungen, 16 action_items und fuenf brechende Aenderungen. Davon trifft fast nichts diese Anwendung - und das ist ein Befund, kein Versaeumnis: - finding_status (brechend, 2.7.0): kein Aufrufer. Weder /api/supervisor/v1/observe noch /status werden hier angesprochen. Vorgemerkt fuer #528 Stufe 3 - wer den Manager baut, liest den Befundzustand von Anfang an aus diesem Feld. - Nachpruefungs-Status awaiting_verification, in_verification, fix_insufficient (brechend, 2.6.1): Die Anwendung wertet bugtracker_items.status nirgends aus. Betrifft den Arbeitsablauf der Agenten, und der steht in CLAUDE.md bereits richtig beschrieben. - Meldungen ohne project_slug (brechend, 2.8.1): BugtrackerClient und ErrorReporter schicken project_slug und build_version seit jeher mit. - Leseweg ohne project_slug und schmales dc_roadmap (brechend, 2.8.1): betrifft den MCP-Adapter, nicht den Anwendungscode. Geblieben ist die eine Zusage, die man nicht sieht, wenn man sie verletzt: Seit 2.8.1 erbt ein Watchdog-Monitor das Projekt seines Ping-Tokens. Ohne Bindung landen die Befunde im Scheinprojekt "default", und eine Abfrage nach dem eigenen Projekt liefert fuer eine laufende, fleissig meldende Instanz null Befunde - die Aufsicht sieht aus wie "nichts auffaellig". Das ist genau die Falle, in die hier schon einmal Meldungen gelaufen sind. Deshalb neu: TokenProvisioner.WhoAmIAsync gegen GET /api/tokens/v1/whoami (SDK 2.7.1) und der Typ TokenIdentity mit IsProjectBound. DeploymentcenterService fragt beim Start das eigene Melde-Token, protokolliert das Projekt und warnt, wenn keines dransteht. Bewusst nur eine Warnung: Monitoring, das bei unklarer Rechtelage gar nicht erst anlaeuft, ist genau dann still, wenn man es braucht. Faellt die Auskunft aus - aelterer Server, kein Netz -, bleibt es bei Debug. Gemessen statt geraten: TokenProvisioner schickt keinen project_slug mit, und ob der Server die Bindung vom Master-Token erbt, steht in keiner Beschreibung, die vorliegt. Ein Sub-Token dafuer eigens auf dem Produktivserver auszustellen waere zu viel Nebenwirkung gewesen; die erste Instanz, die eines zieht, sagt es jetzt selbst ins Protokoll. #176 ist erledigt und die Regel in CLAUDE.md gestrichen: dc_roadmap ohne project_slug liefert jetzt das eigene Projekt (nachgemessen: 51 Eintraege, alle clawddotnet). Wer projektuebergreifend lesen will, schickt project_slug=all. dc_link_project auf 2.8.2 gesetzt, CLAUDE.md-Zeile hochgesetzt. Build ohne Warnungen, 717 Tests gruen, 8 uebersprungen. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
Richard
@@ -121,6 +121,8 @@ public sealed class DeploymentcenterService : IAsyncDisposable
|
||||
var watchdog = instance.Watchdog;
|
||||
var token = await ResolveInstanceTokenAsync(instance, saveInstanceConfig, ct);
|
||||
|
||||
await WarnIfTokenHasNoProjectAsync(token, ct);
|
||||
|
||||
try
|
||||
{
|
||||
_heartbeat = WatchdogHeartbeatService.Create(
|
||||
@@ -191,6 +193,54 @@ public sealed class DeploymentcenterService : IAsyncDisposable
|
||||
}
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Prüft, ob das Melde-Token an ein Projekt gebunden ist, und sagt es, wenn nicht.
|
||||
///
|
||||
/// <para><b>Warum das eine eigene Anfrage wert ist.</b> Seit SDK 2.8.1 erbt ein
|
||||
/// Watchdog-Monitor das Projekt seines Ping-Tokens. Fehlt dort die Bindung, legt der
|
||||
/// Server die Befunde im Scheinprojekt „default" ab — und eine Abfrage nach dem
|
||||
/// eigenen Projekt liefert für eine laufende, fleißig meldende Instanz dann
|
||||
/// <b>null Befunde</b>. Die Aufsicht sieht aus wie „nichts auffällig", und genau
|
||||
/// darauf verlässt sich niemand gern.</para>
|
||||
///
|
||||
/// <para>Der Fall ist nicht erfunden: Genau so sind schon Meldungen in „default"
|
||||
/// gelandet und waren danach für ihren eigenen Melder gesperrt. Ein Heartbeat, der
|
||||
/// ins Leere geht, meldet sich nicht von selbst — deshalb wird hier einmal
|
||||
/// nachgefragt, statt es anzunehmen.</para>
|
||||
///
|
||||
/// <para>Nur eine Warnung, kein Abbruch: Ein Monitoring, das bei unklarer Rechtelage
|
||||
/// gar nicht erst anläuft, ist genau dann still, wenn man es braucht.</para>
|
||||
/// </summary>
|
||||
private async Task WarnIfTokenHasNoProjectAsync(string token, CancellationToken ct)
|
||||
{
|
||||
try
|
||||
{
|
||||
using var api = new DeploymentcenterApi(_api.BaseUrl, token);
|
||||
var identity = await new TokenProvisioner(api).WhoAmIAsync(ct);
|
||||
|
||||
if (identity.IsProjectBound)
|
||||
{
|
||||
_logger.LogInformation(
|
||||
"Deploymentcenter: Meldungen gehen an Projekt {Projekt} (Token {TokenId}, {Typ}).",
|
||||
identity.ProjectSlug, identity.TokenId, identity.Type);
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
_logger.LogWarning(
|
||||
"Deploymentcenter: Das Melde-Token {TokenId} ist an kein Projekt gebunden. "
|
||||
+ "Watchdog-Befunde dieser Instanz landen damit im Scheinprojekt \"default\" "
|
||||
+ "und tauchen in der Projektansicht nicht auf.",
|
||||
identity.TokenId);
|
||||
}
|
||||
catch (Exception ex)
|
||||
{
|
||||
// Aelterer Server ohne whoami (vor SDK 2.7.1), kein Netz, kein Recht: Das ist
|
||||
// eine Auskunft, die ausfaellt, und kein Grund, den Heartbeat nicht zu starten.
|
||||
_logger.LogDebug(ex, "Deploymentcenter: Token-Auskunft nicht verfügbar.");
|
||||
}
|
||||
}
|
||||
|
||||
private static string DescribePlatform() =>
|
||||
$"{System.Runtime.InteropServices.RuntimeInformation.OSDescription} / "
|
||||
+ $".NET {System.Environment.Version}";
|
||||
|
||||
@@ -9,8 +9,30 @@ namespace ClawdDotNet.Core.Deploymentcenter;
|
||||
public sealed record ProvisionedToken(string Token, string TokenId, IReadOnlyList<string> Scopes);
|
||||
|
||||
/// <summary>
|
||||
/// Tauscht das anwendungsweite Master-Token gegen ein eigenes Sub-Token je Instanz
|
||||
/// (<c>POST /api/tokens/v1/provision</c>).
|
||||
/// Was ein Token über sich selbst sagt (<c>GET /api/tokens/v1/whoami</c>, seit SDK 2.7.1).
|
||||
///
|
||||
/// <para><see cref="ProjectSlug"/> ist das Feld, auf das es ankommt. Seit SDK 2.8.1 erbt
|
||||
/// ein Watchdog-Monitor das Projekt seines Ping-Tokens — ist dort nichts eingetragen,
|
||||
/// landen die Befunde im Scheinprojekt „default", und eine Abfrage nach dem eigenen
|
||||
/// Projekt liefert für eine laufende, meldende Instanz <b>null Befunde</b>. Die Aufsicht
|
||||
/// sähe dann aus wie „nichts auffällig", und das ist der schlechteste aller
|
||||
/// Irrtümer.</para>
|
||||
/// </summary>
|
||||
public sealed record TokenIdentity(
|
||||
string TokenId,
|
||||
string Name,
|
||||
string Type,
|
||||
string? ProjectSlug,
|
||||
string Environment,
|
||||
IReadOnlyList<string> Scopes)
|
||||
{
|
||||
/// <summary>Ob dieses Token an ein Projekt gebunden ist — siehe Klassenkommentar.</summary>
|
||||
public bool IsProjectBound => !string.IsNullOrWhiteSpace(ProjectSlug);
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Der Zugang zu <c>/api/tokens/v1/…</c>: ein Sub-Token je Instanz ausstellen
|
||||
/// (<c>provision</c>) und ein Token nach sich selbst fragen (<c>whoami</c>).
|
||||
///
|
||||
/// <para>Das ersetzt die frühere Selbstregistrierung über <c>POST /api/register</c> —
|
||||
/// diesen Endpunkt gibt es im Deploymentcenter nicht (und im alten WatchDog-Server war
|
||||
@@ -70,4 +92,39 @@ public sealed class TokenProvisioner(DeploymentcenterApi api)
|
||||
|
||||
return new ProvisionedToken(token, tokenId, granted);
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Fragt das Token, mit dem diese Verbindung arbeitet, nach sich selbst.
|
||||
///
|
||||
/// <para>Nicht Neugier, sondern die einzige Möglichkeit, die Projektbindung zu
|
||||
/// erfahren: Sie steht nicht im Token und ergibt sich nicht aus dem, was beim
|
||||
/// Ausstellen mitgeschickt wurde — sie ist eine Eigenschaft, die der Server führt.
|
||||
/// Siehe <see cref="TokenIdentity"/>, warum das nicht gleichgültig ist.</para>
|
||||
/// </summary>
|
||||
public async Task<TokenIdentity> WhoAmIAsync(CancellationToken ct = default)
|
||||
{
|
||||
var response = await api.GetAsync("/api/tokens/v1/whoami", ct).ConfigureAwait(false);
|
||||
|
||||
var scopes = new List<string>();
|
||||
if (response.TryGetProperty("scopes", out var s) && s.ValueKind == JsonValueKind.Array)
|
||||
{
|
||||
scopes.AddRange(s.EnumerateArray()
|
||||
.Where(e => e.ValueKind == JsonValueKind.String)
|
||||
.Select(e => e.GetString()!));
|
||||
}
|
||||
|
||||
return new TokenIdentity(
|
||||
Text(response, "token_id") ?? "",
|
||||
Text(response, "name") ?? "",
|
||||
Text(response, "type") ?? "unknown",
|
||||
Text(response, "project_slug"),
|
||||
Text(response, "environment") ?? "",
|
||||
scopes);
|
||||
}
|
||||
|
||||
/// <summary>Eine Zeichenkette aus der Antwort; <c>null</c> auch bei JSON-<c>null</c>.</summary>
|
||||
private static string? Text(JsonElement root, string name)
|
||||
=> root.TryGetProperty(name, out var v) && v.ValueKind == JsonValueKind.String
|
||||
? v.GetString()
|
||||
: null;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user