Merge branch 'main' of https://git.stella-ops.org/stella-ops.org/git.stella-ops.org
This commit is contained in:
@@ -15,6 +15,7 @@ using StellaOps.Concelier.Storage;
|
||||
using StellaOps.Concelier.Storage.Advisories;
|
||||
using StellaOps.Concelier.Storage;
|
||||
using StellaOps.Concelier.Storage;
|
||||
using StellaOps.Concelier.Core.Canonical;
|
||||
using StellaOps.Plugin;
|
||||
|
||||
namespace StellaOps.Concelier.Connector.Ghsa;
|
||||
@@ -37,6 +38,7 @@ public sealed class GhsaConnector : IFeedConnector
|
||||
private readonly GhsaDiagnostics _diagnostics;
|
||||
private readonly TimeProvider _timeProvider;
|
||||
private readonly ILogger<GhsaConnector> _logger;
|
||||
private readonly ICanonicalAdvisoryService? _canonicalService;
|
||||
private readonly object _rateLimitWarningLock = new();
|
||||
private readonly Dictionary<(string Phase, string Resource), bool> _rateLimitWarnings = new();
|
||||
|
||||
@@ -50,7 +52,8 @@ public sealed class GhsaConnector : IFeedConnector
|
||||
IOptions<GhsaOptions> options,
|
||||
GhsaDiagnostics diagnostics,
|
||||
TimeProvider? timeProvider,
|
||||
ILogger<GhsaConnector> logger)
|
||||
ILogger<GhsaConnector> logger,
|
||||
ICanonicalAdvisoryService? canonicalService = null)
|
||||
{
|
||||
_fetchService = fetchService ?? throw new ArgumentNullException(nameof(fetchService));
|
||||
_rawDocumentStorage = rawDocumentStorage ?? throw new ArgumentNullException(nameof(rawDocumentStorage));
|
||||
@@ -63,6 +66,7 @@ public sealed class GhsaConnector : IFeedConnector
|
||||
_diagnostics = diagnostics ?? throw new ArgumentNullException(nameof(diagnostics));
|
||||
_timeProvider = timeProvider ?? TimeProvider.System;
|
||||
_logger = logger ?? throw new ArgumentNullException(nameof(logger));
|
||||
_canonicalService = canonicalService; // Optional - canonical ingest
|
||||
}
|
||||
|
||||
public string SourceName => GhsaConnectorPlugin.SourceName;
|
||||
@@ -399,6 +403,14 @@ public sealed class GhsaConnector : IFeedConnector
|
||||
|
||||
await _advisoryStore.UpsertAsync(advisory, cancellationToken).ConfigureAwait(false);
|
||||
await _documentStore.UpdateStatusAsync(document.Id, DocumentStatuses.Mapped, cancellationToken).ConfigureAwait(false);
|
||||
|
||||
// Ingest to canonical advisory service if available
|
||||
if (_canonicalService is not null)
|
||||
{
|
||||
var rawPayloadJson = dtoRecord.Payload.ToJson();
|
||||
await IngestToCanonicalAsync(advisory, rawPayloadJson, document.FetchedAt, cancellationToken).ConfigureAwait(false);
|
||||
}
|
||||
|
||||
pendingMappings.Remove(documentId);
|
||||
_diagnostics.MapSuccess(1);
|
||||
}
|
||||
@@ -544,4 +556,90 @@ public sealed class GhsaConnector : IFeedConnector
|
||||
|
||||
return false;
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Ingests GHSA advisory to canonical advisory service for deduplication.
|
||||
/// Creates one RawAdvisory per affected package.
|
||||
/// </summary>
|
||||
private async Task IngestToCanonicalAsync(
|
||||
Advisory advisory,
|
||||
string rawPayloadJson,
|
||||
DateTimeOffset fetchedAt,
|
||||
CancellationToken cancellationToken)
|
||||
{
|
||||
if (_canonicalService is null || advisory.AffectedPackages.IsEmpty)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
// Find primary CVE from aliases
|
||||
var cve = advisory.Aliases
|
||||
.FirstOrDefault(a => a.StartsWith("CVE-", StringComparison.OrdinalIgnoreCase))
|
||||
?? advisory.AdvisoryKey; // Fall back to GHSA ID if no CVE
|
||||
|
||||
// Extract CWE weaknesses
|
||||
var weaknesses = advisory.Cwes
|
||||
.Where(w => w.Identifier.StartsWith("CWE-", StringComparison.OrdinalIgnoreCase))
|
||||
.Select(w => w.Identifier)
|
||||
.ToList();
|
||||
|
||||
// Create one RawAdvisory per affected package
|
||||
foreach (var affected in advisory.AffectedPackages)
|
||||
{
|
||||
if (string.IsNullOrWhiteSpace(affected.Identifier))
|
||||
{
|
||||
continue;
|
||||
}
|
||||
|
||||
// Build version range JSON
|
||||
string? versionRangeJson = null;
|
||||
if (!affected.VersionRanges.IsEmpty)
|
||||
{
|
||||
var firstRange = affected.VersionRanges[0];
|
||||
var rangeObj = new
|
||||
{
|
||||
introduced = firstRange.IntroducedVersion,
|
||||
@fixed = firstRange.FixedVersion,
|
||||
last_affected = firstRange.LastAffectedVersion
|
||||
};
|
||||
versionRangeJson = JsonSerializer.Serialize(rangeObj, SerializerOptions);
|
||||
}
|
||||
|
||||
var rawAdvisory = new RawAdvisory
|
||||
{
|
||||
SourceAdvisoryId = advisory.AdvisoryKey,
|
||||
Cve = cve,
|
||||
AffectsKey = affected.Identifier,
|
||||
VersionRangeJson = versionRangeJson,
|
||||
Weaknesses = weaknesses,
|
||||
PatchLineage = null,
|
||||
Severity = advisory.Severity,
|
||||
Title = advisory.Title,
|
||||
Summary = advisory.Summary,
|
||||
VendorStatus = VendorStatus.Affected,
|
||||
RawPayloadJson = rawPayloadJson,
|
||||
FetchedAt = fetchedAt
|
||||
};
|
||||
|
||||
try
|
||||
{
|
||||
var result = await _canonicalService.IngestAsync(SourceName, rawAdvisory, cancellationToken).ConfigureAwait(false);
|
||||
|
||||
if (_logger.IsEnabled(LogLevel.Debug))
|
||||
{
|
||||
_logger.LogDebug(
|
||||
"Canonical ingest for {GhsaId}/{AffectsKey}: {Decision} (canonical={CanonicalId})",
|
||||
advisory.AdvisoryKey, affected.Identifier, result.Decision, result.CanonicalId);
|
||||
}
|
||||
}
|
||||
catch (Exception ex)
|
||||
{
|
||||
_logger.LogWarning(
|
||||
ex,
|
||||
"Failed to ingest {GhsaId}/{AffectsKey} to canonical service",
|
||||
advisory.AdvisoryKey, affected.Identifier);
|
||||
// Don't fail the mapping operation for canonical ingest failures
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
@@ -13,9 +13,8 @@ using StellaOps.Concelier.Connector.Nvd.Configuration;
|
||||
using StellaOps.Concelier.Connector.Nvd.Internal;
|
||||
using StellaOps.Concelier.Storage;
|
||||
using StellaOps.Concelier.Storage.Advisories;
|
||||
using StellaOps.Concelier.Storage;
|
||||
using StellaOps.Concelier.Storage;
|
||||
using StellaOps.Concelier.Storage.ChangeHistory;
|
||||
using StellaOps.Concelier.Storage.Contracts;
|
||||
using StellaOps.Concelier.Core.Canonical;
|
||||
using StellaOps.Plugin;
|
||||
using Json.Schema;
|
||||
@@ -322,7 +321,7 @@ public sealed class NvdConnector : IFeedConnector
|
||||
private async Task<IReadOnlyCollection<Guid>> FetchAdditionalPagesAsync(
|
||||
TimeWindow window,
|
||||
IReadOnlyDictionary<string, string> baseMetadata,
|
||||
DocumentRecord firstDocument,
|
||||
StorageDocument firstDocument,
|
||||
CancellationToken cancellationToken)
|
||||
{
|
||||
if (firstDocument.PayloadId is null)
|
||||
|
||||
@@ -52,13 +52,13 @@ public sealed class EwsVerdictDeterminismTests
|
||||
// Assert - All results should be byte-identical
|
||||
var firstScore = results[0].Score;
|
||||
var firstBucket = results[0].Bucket;
|
||||
var firstDimensions = results[0].Dimensions;
|
||||
var firstBreakdown = results[0].Breakdown;
|
||||
|
||||
results.Should().AllSatisfy(r =>
|
||||
{
|
||||
r.Score.Should().Be(firstScore, "score must be deterministic");
|
||||
r.Bucket.Should().Be(firstBucket, "bucket must be deterministic");
|
||||
r.Dimensions.Should().BeEquivalentTo(firstDimensions, "dimensions must be deterministic");
|
||||
r.Breakdown.Should().BeEquivalentTo(firstBreakdown, "breakdown must be deterministic");
|
||||
});
|
||||
}
|
||||
|
||||
@@ -155,16 +155,16 @@ public sealed class EwsVerdictDeterminismTests
|
||||
// Custom policy with different weights
|
||||
var customPolicy = new EvidenceWeightPolicy
|
||||
{
|
||||
PolicyId = "custom-test-policy",
|
||||
Version = "1.0",
|
||||
Version = "ews.v1",
|
||||
Profile = "custom-test-policy",
|
||||
Weights = new EvidenceWeights
|
||||
{
|
||||
Reachability = 0.50, // Much higher weight on reachability
|
||||
Runtime = 0.10,
|
||||
Backport = 0.05,
|
||||
Exploit = 0.20,
|
||||
Source = 0.10,
|
||||
Mitigation = 0.05
|
||||
Rch = 0.50, // Much higher weight on reachability
|
||||
Rts = 0.10,
|
||||
Bkp = 0.05,
|
||||
Xpl = 0.20,
|
||||
Src = 0.10,
|
||||
Mit = 0.05
|
||||
},
|
||||
Buckets = EvidenceWeightPolicy.DefaultProduction.Buckets
|
||||
};
|
||||
|
||||
@@ -189,17 +189,17 @@ public sealed class RevocationLedgerTests
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void Clear_RemovesAllEntries()
|
||||
public async Task Clear_RemovesAllEntries()
|
||||
{
|
||||
// Arrange
|
||||
_ledger.RecordAsync(CreateTestEntry(RevocationTypes.Signer, "s1")).GetAwaiter().GetResult();
|
||||
_ledger.RecordAsync(CreateTestEntry(RevocationTypes.Signer, "s2")).GetAwaiter().GetResult();
|
||||
await _ledger.RecordAsync(CreateTestEntry(RevocationTypes.Signer, "s1"));
|
||||
await _ledger.RecordAsync(CreateTestEntry(RevocationTypes.Signer, "s2"));
|
||||
|
||||
// Act
|
||||
_ledger.Clear();
|
||||
|
||||
// Assert
|
||||
var seqNo = _ledger.GetLatestSeqNoAsync().GetAwaiter().GetResult();
|
||||
var seqNo = await _ledger.GetLatestSeqNoAsync();
|
||||
seqNo.Should().Be(0);
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user