v0.29.10 — Admin UI refonte (modals + onglets), emails de release, 4-digit versions
Admin backoffice — UI redesign : - Pages versions.php + licenses.php : remplace les <details> inline qui débordaient horizontalement par UN bouton « ✎ Modifier » par row qui ouvre une modal <dialog> avec onglets. 5 tabs versions (Méta, Notes, BÊTA, Channels, Avancé) + 6 tabs licenses (Prolonger, Slots, Channel, BÊTA, Lock, Contacts, Machines). Délégation JS unique pour les onglets. - Bouton 📋 Copier la clé license dans chaque row (Clipboard API + feedback visuel ✓ vert 1.5s). Évite le détour par phpMyAdmin pour transmettre la clé aux clients. - Overlay « hashing en cours » plein écran sur tous les boutons de hash (3-5 min sur OVH pour 13 Go ZIP). Spinner CSS + message contextualisé par scope (bulk vs single). - Date de release passée de datetime-local à date (l'heure n'a pas de sens UX), avec défaut = aujourd'hui pour release_at et aujourd'hui-1an pour min_license_date (= license standard couvre les releases sur 1 an). Emails de notification release : - Migration 004 : colonne contact_emails TEXT NULL sur licenses (CSV) - Onglet « Contacts » sur la modal licenses pour saisir les emails par license (parsing tolérant : CSV, ligne par ligne, point-virgule) - Bouton « ✉ Notifier » par version : POST notify_release filtre les licenses éligibles (channel match + min_license_date + can_see_betas pour BÊTA) et envoie un email HTML à chaque contact (dédup global) - Template email table-based + bgcolor (compat Outlook/Word engine), navy foncé #0F172A, logo Asterion en CID embed (= affichage direct sans demande de permission Outlook), bouton download installer centré (align="center" + margin auto), release notes en <pre> - Mailer.php helper : parse emails, multipart/related avec attachments inline, fallback execCommand pour clipboard 4-digit version support (X.Y.Z.B) : - SemVer Parse/CompareTo/ToString gèrent 3 ou 4 digits ; Build absent = 0 implicite (1.5.4 == 1.5.4.0 < 1.5.4.13). HasExplicitBuild préserve le format d'origine au round-trip. - Regex InstallationRegistry étendue avec (?:\.\d+)? → reconnaît « PROSERVE v1.5.4.13 » côte-à-côte avec « PROSERVE v1.5.4 » sur disque - Server-side : versions.php, launcher.php, DownloadUrl.php, api/index.php, Releasenotes.php — toutes les regex de validation acceptent le 4ᵉ digit - Use case : dev/test iterations cohabitant avec leur release stable Bugs fixes : - migrate.php : strip ligne par ligne les commentaires SQL avant le check is-empty. Sans ça, le PREMIER chunk d'un fichier migration (= header + premier ALTER) commençait par `--` et était silencieusement skip → ALTER jamais appliqué. Affectait migrations 003, 004. - SignManifest::getOrComputeSha256 ignorait son cache interne quand force demandé par le caller, retournant l'ancien hash en 0 ms même après re-upload SFTP (avec mtime préservé). Propage maintenant le flag $force. Bouton « 🔁 Hash » per-row force maintenant un re-calcul systématique. - DownloadManager : 416 (Range Not Satisfiable) ajouté aux URL-refresh triggers, avec HEAD probe pour comparer taille serveur vs manifest → message d'erreur explicite si ZIP tronqué. Bps display lissé sur une fenêtre glissante de 12 samples (3 s) → plus de clignotement quand un segment finit / Polly retry. SHA mismatch popup enrichi avec les deux SHAs (attendu vs calculé) extraits via regex de l'exception. - DownloadUrl.php : signature de l'URL utilisait un template hardcodé /builds/proserve-{version}.zip, ignorant tout rename serveur. Lit maintenant download.url du manifest et signe le filename réel. Strings i18n (5 langues) : - ~15 nouveaux : SHA mismatch enrichi avec sources, 416 size mismatch, stale manifest, manifest refreshed auto-retry, force fresh menu Bumps : 0.29.7 → 0.29.10 (4-digit support + accumulated UI fixes). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -312,14 +312,26 @@ public sealed class DownloadManager : IDownloadManager
|
||||
// Reporter dédié : toutes les 250 ms, snapshot le compteur, calcule bps/ETA, reporte.
|
||||
// Tous les 5 s ou 50 Mo, persist le state.json.
|
||||
// Tourne sur un thread séparé pour ne jamais être starvé par les workers DL.
|
||||
//
|
||||
// Calcul du débit : on utilise une FENÊTRE GLISSANTE de ~3 s (12 samples
|
||||
// de 250ms) plutôt qu'un delta instantané sur le dernier tick. Pourquoi :
|
||||
// avec un delta tick-à-tick, dès qu'un segment finit OU que Polly retry
|
||||
// entre 2 tentatives OU qu'il y a un micro-stall réseau, le compteur ne
|
||||
// bouge pas pendant 1-2 ticks → bps = 0 → ETA = null → UI cache les
|
||||
// deux 250-500 ms → clignotement permanent à l'écran (symptôme rapporté).
|
||||
// Avec la fenêtre, un trou de 1-2 ticks est dilué dans 12 samples → le
|
||||
// débit affiché reste stable. La latence d'adaptation est ~3s, ce qui
|
||||
// est largement OK visuellement (l'opérateur ne perçoit pas 3s de retard
|
||||
// sur un chiffre qui de toute façon fluctue de ±5-10 % au cours d'un DL).
|
||||
const int BpsWindowSize = 12; // 12 × 250 ms = 3 s
|
||||
using var reporterCts = CancellationTokenSource.CreateLinkedTokenSource(ct);
|
||||
var reporterTask = Task.Run(async () =>
|
||||
{
|
||||
var sw = Stopwatch.StartNew();
|
||||
var lastReport = TimeSpan.Zero;
|
||||
long lastReportBytes = Interlocked.Read(ref aggregateBytes);
|
||||
var lastFlush = TimeSpan.Zero;
|
||||
long lastFlushBytes = lastReportBytes;
|
||||
long lastFlushBytes = Interlocked.Read(ref aggregateBytes);
|
||||
// Fenêtre glissante (time, bytes) pour lissage du débit.
|
||||
var bpsWindow = new Queue<(TimeSpan At, long Bytes)>(BpsWindowSize + 1);
|
||||
try
|
||||
{
|
||||
while (!reporterCts.IsCancellationRequested)
|
||||
@@ -327,13 +339,24 @@ public sealed class DownloadManager : IDownloadManager
|
||||
await Task.Delay(250, reporterCts.Token).ConfigureAwait(false);
|
||||
var snapshot = Interlocked.Read(ref aggregateBytes);
|
||||
var elapsed = sw.Elapsed;
|
||||
var deltaSec = (elapsed - lastReport).TotalSeconds;
|
||||
var bps = deltaSec > 0 ? (snapshot - lastReportBytes) / deltaSec : 0;
|
||||
|
||||
// Push le nouveau sample, évince le plus ancien si on dépasse.
|
||||
bpsWindow.Enqueue((elapsed, snapshot));
|
||||
while (bpsWindow.Count > BpsWindowSize) bpsWindow.Dequeue();
|
||||
|
||||
// bps = (bytes_now - bytes_oldest_in_window) / time_span_window
|
||||
// Fallback à 0 tant qu'on n'a pas au moins 2 samples (le 1er tick).
|
||||
double bps = 0;
|
||||
if (bpsWindow.Count >= 2)
|
||||
{
|
||||
var oldest = bpsWindow.Peek();
|
||||
var span = (elapsed - oldest.At).TotalSeconds;
|
||||
if (span > 0) bps = (snapshot - oldest.Bytes) / span;
|
||||
}
|
||||
TimeSpan? eta = null;
|
||||
if (bps > 0 && total > snapshot) eta = TimeSpan.FromSeconds((total - snapshot) / bps);
|
||||
if (bps > 0 && total > snapshot)
|
||||
eta = TimeSpan.FromSeconds((total - snapshot) / bps);
|
||||
progress?.Report(new DownloadProgress(snapshot, total, bps, eta));
|
||||
lastReport = elapsed;
|
||||
lastReportBytes = snapshot;
|
||||
|
||||
if ((elapsed - lastFlush).TotalSeconds >= StateFlushSecondsInterval ||
|
||||
snapshot - lastFlushBytes >= StateFlushBytesInterval)
|
||||
@@ -475,7 +498,8 @@ public sealed class DownloadManager : IDownloadManager
|
||||
// un faux 404 fatal sur tous les segments sauf le premier.
|
||||
bool isUrlRefreshTrigger = resp.StatusCode == HttpStatusCode.Forbidden
|
||||
|| resp.StatusCode == HttpStatusCode.Gone
|
||||
|| resp.StatusCode == HttpStatusCode.NotFound;
|
||||
|| resp.StatusCode == HttpStatusCode.NotFound
|
||||
|| resp.StatusCode == HttpStatusCode.RequestedRangeNotSatisfiable;
|
||||
if (isUrlRefreshTrigger && job.RefreshUrlAsync is not null)
|
||||
{
|
||||
await GetOrRefreshUrlAsync(job, forceRefresh: true, ct).ConfigureAwait(false);
|
||||
@@ -499,6 +523,37 @@ public sealed class DownloadManager : IDownloadManager
|
||||
$"HTTP 404 on segment {seg.Index} — le fichier a été renommé/supprimé côté serveur et le manifest local est obsolète. Clique « Vérifier les MAJ » avant de réessayer.",
|
||||
isTransient: false);
|
||||
}
|
||||
// 416 unchanged URL = vrai mismatch de taille. On fait un HEAD probe
|
||||
// pour obtenir la taille RÉELLE côté serveur et la rapporter dans le
|
||||
// message d'erreur — l'opérateur saura ainsi s'il a un upload SFTP
|
||||
// tronqué (file plus petit que prévu) ou un manifest qui ment sur
|
||||
// sizeBytes. Cas concret : SFTP coupe au milieu d'un upload de 13 Go
|
||||
// → le file fait 7 Go sur disque, mais le manifest dit 13 Go.
|
||||
// Segment N qui demande bytes 9G-10G reçoit 416 (out of range).
|
||||
if (resp.StatusCode == HttpStatusCode.RequestedRangeNotSatisfiable)
|
||||
{
|
||||
long? actualSize = null;
|
||||
try
|
||||
{
|
||||
using var headReq = new HttpRequestMessage(HttpMethod.Head, url);
|
||||
using var headResp = await _http.SendAsync(headReq, HttpCompletionOption.ResponseHeadersRead, ct).ConfigureAwait(false);
|
||||
if (headResp.IsSuccessStatusCode)
|
||||
actualSize = headResp.Content.Headers.ContentLength;
|
||||
}
|
||||
catch (Exception ex) { _logger.LogDebug(ex, "HEAD probe after 416 failed"); }
|
||||
|
||||
var actualStr = actualSize.HasValue ? $"{actualSize.Value:N0}" : "?";
|
||||
var expectedStr = $"{state.TotalBytes:N0}";
|
||||
_logger.LogError(
|
||||
"HTTP 416 on segment {Seg} — actual server file size: {Actual} bytes, manifest expects: {Expected} bytes",
|
||||
seg.Index, actualStr, expectedStr);
|
||||
throw new HttpResumableException(
|
||||
$"HTTP 416 on segment {seg.Index} — incohérence taille de fichier serveur : " +
|
||||
$"le ZIP sur le serveur fait {actualStr} octets, le manifest attend {expectedStr} octets. " +
|
||||
$"Probable cause : upload SFTP tronqué OU manifest avec sizeBytes/sha256 obsolètes. " +
|
||||
$"Re-upload le ZIP complet puis « 🔁 Hasher les versions + signer » côté admin.",
|
||||
isTransient: false);
|
||||
}
|
||||
throw new HttpResumableException(
|
||||
$"Signed URL expired (HTTP {(int)resp.StatusCode}), refreshed for next try", isTransient: true);
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user