Event export serializer (#3543)

* Release 8.7.1 (hotfix) (#3459)

* Remove the retention policy setting (#3452)

---------

Co-authored-by: Cheick Keita <chkeita@microsoft.com>

* Revert "Release 8.7.1 (hotfix) (#3459)" (#3468)

This reverts commit c69deed50e.

* Redo 8.7.1 (#3469)

* Redo-8.7.1-hotfix

---------

Co-authored-by: Cheick Keita <chkeita@microsoft.com>

* Support custom ado fields that mark work items as duplicate (#3467)

* Add field to ado config for checking duplicate work items

* Make duplicate fields nullable and add it to python models

* Update broken tests

* Update docs to include new ado_duplicate_fields property

* Update readme with archive message (#3408)

Co-authored-by: Adam <103067949+AdamL-Microsoft@users.noreply.github.com>

* Bump tokio from 1.30.0 to 1.32.0 in /src/proxy-manager (#3425)

Bumps [tokio](https://github.com/tokio-rs/tokio) from 1.30.0 to 1.32.0.
- [Release notes](https://github.com/tokio-rs/tokio/releases)
- [Commits](https://github.com/tokio-rs/tokio/compare/tokio-1.30.0...tokio-1.32.0)

---
updated-dependencies:
- dependency-name: tokio
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump tokio from 1.30.0 to 1.32.0 in /src/agent (#3424)

Bumps [tokio](https://github.com/tokio-rs/tokio) from 1.30.0 to 1.32.0.
- [Release notes](https://github.com/tokio-rs/tokio/releases)
- [Commits](https://github.com/tokio-rs/tokio/compare/tokio-1.30.0...tokio-1.32.0)

---
updated-dependencies:
- dependency-name: tokio
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Remove unnecessary method argument (#3473)

* Bump elsa from 1.8.1 to 1.9.0 in /src/agent (#3411)

Bumps [elsa](https://github.com/manishearth/elsa) from 1.8.1 to 1.9.0.
- [Commits](https://github.com/manishearth/elsa/compare/v1.8.1...v1.9.0)

---
updated-dependencies:
- dependency-name: elsa
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump tempfile from 3.7.1 to 3.8.0 in /src/agent (#3437)

Bumps [tempfile](https://github.com/Stebalien/tempfile) from 3.7.1 to 3.8.0.
- [Changelog](https://github.com/Stebalien/tempfile/blob/master/CHANGELOG.md)
- [Commits](https://github.com/Stebalien/tempfile/compare/v3.7.1...v3.8.0)

---
updated-dependencies:
- dependency-name: tempfile
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump tempfile from 3.7.1 to 3.8.0 in /src/proxy-manager (#3436)

Bumps [tempfile](https://github.com/Stebalien/tempfile) from 3.7.1 to 3.8.0.
- [Changelog](https://github.com/Stebalien/tempfile/blob/master/CHANGELOG.md)
- [Commits](https://github.com/Stebalien/tempfile/compare/v3.7.1...v3.8.0)

---
updated-dependencies:
- dependency-name: tempfile
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Updating requirements.txt to accept >= onefuzztypes. (#3477)

* Updating requirements.txt to accept >= onefuzztypes.

* Trying to loosen restriction.

* Bump notify from 6.0.1 to 6.1.1 in /src/agent (#3435)

Bumps [notify](https://github.com/notify-rs/notify) from 6.0.1 to 6.1.1.
- [Release notes](https://github.com/notify-rs/notify/releases)
- [Changelog](https://github.com/notify-rs/notify/blob/main/CHANGELOG.md)
- [Commits](https://github.com/notify-rs/notify/compare/notify-6.0.1...notify-6.1.1)

---
updated-dependencies:
- dependency-name: notify
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump azure_* crates (#3478)

* Release 8.8.0 (#3466)

* Release 8.8.0

* Bump clap from 4.3.21 to 4.4.2 in /src/agent (#3484)

Bumps [clap](https://github.com/clap-rs/clap) from 4.3.21 to 4.4.2.
- [Release notes](https://github.com/clap-rs/clap/releases)
- [Changelog](https://github.com/clap-rs/clap/blob/master/CHANGELOG.md)
- [Commits](https://github.com/clap-rs/clap/compare/v4.3.21...v4.4.2)

---
updated-dependencies:
- dependency-name: clap
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump gimli from 0.27.3 to 0.28.0 in /src/agent (#3414)

Bumps [gimli](https://github.com/gimli-rs/gimli) from 0.27.3 to 0.28.0.
- [Changelog](https://github.com/gimli-rs/gimli/blob/master/CHANGELOG.md)
- [Commits](https://github.com/gimli-rs/gimli/compare/0.27.3...0.28.0)

---
updated-dependencies:
- dependency-name: gimli
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump clap from 4.3.21 to 4.4.2 in /src/proxy-manager (#3474)

Bumps [clap](https://github.com/clap-rs/clap) from 4.3.21 to 4.4.2.
- [Release notes](https://github.com/clap-rs/clap/releases)
- [Changelog](https://github.com/clap-rs/clap/blob/master/CHANGELOG.md)
- [Commits](https://github.com/clap-rs/clap/compare/v4.3.21...v4.4.2)

---
updated-dependencies:
- dependency-name: clap
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump winreg from 0.50.0 to 0.51.0 in /src/agent (#3434)

Bumps [winreg](https://github.com/gentoo90/winreg-rs) from 0.50.0 to 0.51.0.
- [Release notes](https://github.com/gentoo90/winreg-rs/releases)
- [Changelog](https://github.com/gentoo90/winreg-rs/blob/master/CHANGELOG.md)
- [Commits](https://github.com/gentoo90/winreg-rs/compare/v0.50.0...v0.51.0)

---
updated-dependencies:
- dependency-name: winreg
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Adam <103067949+AdamL-Microsoft@users.noreply.github.com>

* Starting integration tests (#3438)

* Starting integration tests

* Ready to test the test

* Parametrize test

* checkpoint

* Test works

* Run integration tests in pipeline

* fmt

* .

* -p

* Install clang

* quotes not required in yaml?

* Hopefully fixed windows?

* Try without killondrop

* lint

* small test

* another test

* Reuse core name

* Wrong step

* bump tokio?

* Try with rust

* make build happy

* Bump pete and small clean up

* Clean up and make the test pass regularly

* fix broken ci

* Lower the poll timeout

* Set the timeout in a nicer way

* fix windows

* fmt

* Include and copy pdbs

* Ignore if pdb is missing on linux

* It takes too long for coverage to be generated

* lint

* Only warn on missing coverage since it's flaky

* Fix windows build

* Small clean up

* Try lowering the poll delay

* fix coverage

* PR comments

* .

* Apparently make is missing?

* Remove aggressive step skipping in CI

* Fix sed checks for CLI versioning (#3486)

* Fix sed checks for CLI versioning

* Fix.

* Fix.

* Changing build_cli

* Trying greater than

* Tring once more.

* Trying major minor

* trying to replace major minor

* Using major minor

* Bump bytes from 1.4.0 to 1.5.0 in /src/agent (#3488)

Bumps [bytes](https://github.com/tokio-rs/bytes) from 1.4.0 to 1.5.0.
- [Release notes](https://github.com/tokio-rs/bytes/releases)
- [Changelog](https://github.com/tokio-rs/bytes/blob/master/CHANGELOG.md)
- [Commits](https://github.com/tokio-rs/bytes/compare/v1.4.0...v1.5.0)

---
updated-dependencies:
- dependency-name: bytes
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Improve area/iteration path validation (#3489)

* Add more comprehensive checks and better error messages to area/iteration path validation

* Join invalid chars with space instead of comma

* Make tree path validation more testable

* Add error code for invalid ADO project in config

* Write unit tests for tree path validation

* Format tree path unit tests

* Merge escape character and control character checks and clarify error message

* Improve handling of unexpected breakpoints (#3493)

* Improve handling of unexpected breakpoints

* fmt

* Update azure_* crates (#3503)

* Fuzz coverage recording (#3322)

* Fuzz coverage recording

* Update cargo.toml

* Update src/agent/coverage/fuzz/fuzz_targets/fuzz_target_record_coverage.rs

Co-authored-by: George Pollard <porges@porg.es>

* Fix fuzz

---------

Co-authored-by: George Pollard <porges@porg.es>

* Reporting coverage on task start up (#3502)

* Reporting coverage on task start up

* Moving metric up.

* Remove feature flag from heartbeat metrics. (#3505)

* Update archive notice. (#3507)

* Add onefuzz service version to job created events (#3504)

* Tevoinea/add version checking in local tasks (#3517)

* Compare task version to service version

* Swallow output when looking for appropriate name

* Create directories if they don't exist in the template (#3522)

* Create directories if they don't exist in the template

* fmt

* Support for retention policies on containers (#3501)

- [x] ability to specify a retention period on a container, which applies to newly-created blobs
- [x] specify default retention periods in templates from CLI side 

There's a small breaking change to the Python JobHelper class.

* Bump rayon from 1.7.0 to 1.8.0 in /src/agent (#3520)

Bumps [rayon](https://github.com/rayon-rs/rayon) from 1.7.0 to 1.8.0.
- [Changelog](https://github.com/rayon-rs/rayon/blob/master/RELEASES.md)
- [Commits](https://github.com/rayon-rs/rayon/compare/rayon-core-v1.7.0...rayon-core-v1.8.0)

---
updated-dependencies:
- dependency-name: rayon
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump insta from 1.31.0 to 1.32.0 in /src/agent (#3521)

Bumps [insta](https://github.com/mitsuhiko/insta) from 1.31.0 to 1.32.0.
- [Changelog](https://github.com/mitsuhiko/insta/blob/master/CHANGELOG.md)
- [Commits](https://github.com/mitsuhiko/insta/compare/1.31.0...1.32.0)

---
updated-dependencies:
- dependency-name: insta
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Disable `repro` and `debug` VM CLI commands. (#3494)

* Disable  and  VM CLI commands.

* Formatting.

* More formatting.

* More formatting.

* Removing Repro check.

* Make modules case insenstive on windows (#3527)

* Make modules and coverage allowlist case insensitive on Windows

* Tests and fmt

* PR comments

* fmt

* Debugging missing file coverage

* fmt

* Broken linux test

* Add a case insensitive transformer for better perf

* cargo fix

* Update windows interceptor list (#3528)

* Starting to work on custom converter

* Add slim serializer for events export

* .

* Cleanup and better tests

* Update feature-flags.bicep

Default off

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Adam <103067949+AdamL-Microsoft@users.noreply.github.com>
Co-authored-by: Cheick Keita <chkeita@microsoft.com>
Co-authored-by: Kanan B <32438208+kananb@users.noreply.github.com>
Co-authored-by: Marc Greisen <mgreisen@microsoft.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Noah McGregor Harper <74685766+nharper285@users.noreply.github.com>
Co-authored-by: George Pollard <gpollard@microsoft.com>
Co-authored-by: George Pollard <porges@porg.es>
This commit is contained in:
Teo Voinea
2023-10-10 13:42:45 -04:00
committed by GitHub
parent e7b0dd5d10
commit 2f2a64980f
7 changed files with 337 additions and 19 deletions

View File

@ -9,4 +9,5 @@ public static class FeatureFlagConstants {
public const string EnableDryRunBlobRetention = "EnableDryRunBlobRetention"; public const string EnableDryRunBlobRetention = "EnableDryRunBlobRetention";
public const string EnableWorkItemCreation = "EnableWorkItemCreation"; public const string EnableWorkItemCreation = "EnableWorkItemCreation";
public const string EnableContainerRetentionPolicies = "EnableContainerRetentionPolicies"; public const string EnableContainerRetentionPolicies = "EnableContainerRetentionPolicies";
public const string EnableSlimEventSerialization = "EnableSlimEventSerialization";
} }

View File

@ -35,7 +35,9 @@ namespace Microsoft.OneFuzz.Service {
private readonly IContainers _containers; private readonly IContainers _containers;
private readonly ICreds _creds; private readonly ICreds _creds;
private readonly JsonSerializerOptions _options; private readonly JsonSerializerOptions _options;
private readonly JsonSerializerOptions _optionsSlim;
private readonly JsonSerializerOptions _deserializingFromBlobOptions; private readonly JsonSerializerOptions _deserializingFromBlobOptions;
private readonly IOnefuzzContext _context;
public Events(ILogger<Events> log, IOnefuzzContext context) { public Events(ILogger<Events> log, IOnefuzzContext context) {
_queue = context.Queue; _queue = context.Queue;
@ -47,9 +49,12 @@ namespace Microsoft.OneFuzz.Service {
DefaultIgnoreCondition = JsonIgnoreCondition.WhenWritingNull DefaultIgnoreCondition = JsonIgnoreCondition.WhenWritingNull
}; };
_options.Converters.Add(new RemoveUserInfo()); _options.Converters.Add(new RemoveUserInfo());
_optionsSlim = new JsonSerializerOptions(_options);
_optionsSlim.Converters.Add(new EventExportConverter<DownloadableEventMessage>());
_deserializingFromBlobOptions = new JsonSerializerOptions(EntityConverter.GetJsonSerializerOptions()) { _deserializingFromBlobOptions = new JsonSerializerOptions(EntityConverter.GetJsonSerializerOptions()) {
DefaultIgnoreCondition = JsonIgnoreCondition.WhenWritingNull DefaultIgnoreCondition = JsonIgnoreCondition.WhenWritingNull
}; };
_context = context;
} }
public virtual async Async.Task QueueSignalrEvent(DownloadableEventMessage message) { public virtual async Async.Task QueueSignalrEvent(DownloadableEventMessage message) {
@ -58,7 +63,13 @@ namespace Microsoft.OneFuzz.Service {
("event_id", message.EventId.ToString()) ("event_id", message.EventId.ToString())
}; };
var ev = new SignalREvent("events", new List<DownloadableEventMessage>() { message }); var ev = new SignalREvent("events", new List<DownloadableEventMessage>() { message });
var queueResult = await _queue.QueueObject("signalr-events", ev, StorageType.Config, serializerOptions: _options);
var opts = await _context.FeatureManagerSnapshot.IsEnabledAsync(FeatureFlagConstants.EnableSlimEventSerialization) switch {
true => _optionsSlim,
false => _options
};
var queueResult = await _queue.QueueObject("signalr-events", ev, StorageType.Config, serializerOptions: opts);
if (!queueResult) { if (!queueResult) {
_log.AddTags(tags); _log.AddTags(tags);
@ -155,16 +166,4 @@ namespace Microsoft.OneFuzz.Service {
); );
} }
} }
public class RemoveUserInfo : JsonConverter<UserInfo> {
public override UserInfo? Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options) {
throw new NotSupportedException("reading UserInfo is not supported");
}
public override void Write(Utf8JsonWriter writer, UserInfo value, JsonSerializerOptions options) {
writer.WriteStartObject();
writer.WriteEndObject();
}
}
} }

View File

@ -20,10 +20,15 @@ public interface IWebhookOperations : IOrm<Webhook> {
public class WebhookOperations : Orm<Webhook>, IWebhookOperations { public class WebhookOperations : Orm<Webhook>, IWebhookOperations {
private readonly IHttpClientFactory _httpFactory; private readonly IHttpClientFactory _httpFactory;
private readonly JsonSerializerOptions _options;
private readonly JsonSerializerOptions _optionsSlim;
public WebhookOperations(IHttpClientFactory httpFactory, ILogger<WebhookOperations> log, IOnefuzzContext context) public WebhookOperations(IHttpClientFactory httpFactory, ILogger<WebhookOperations> log, IOnefuzzContext context)
: base(log, context) { : base(log, context) {
_httpFactory = httpFactory; _httpFactory = httpFactory;
_options = EntityConverter.GetJsonSerializerOptions();
_optionsSlim = new JsonSerializerOptions(_options);
_optionsSlim.Converters.Add(new EventExportConverter<WebhookMessage>());
} }
public async Async.Task SendEvent(DownloadableEventMessage eventMessage) { public async Async.Task SendEvent(DownloadableEventMessage eventMessage) {
@ -139,11 +144,15 @@ public class WebhookOperations : Orm<Webhook>, IWebhookOperations {
string data; string data;
var instanceId = await _context.Containers.GetInstanceId(); var instanceId = await _context.Containers.GetInstanceId();
var webhookMessage = new WebhookMessage(WebhookId: webhookId, EventId: eventId, EventType: eventType, Event: webhookEvent, InstanceId: instanceId, InstanceName: _context.Creds.GetInstanceName(), CreatedAt: eventData.CreatedAt, SasUrl: eventData.SasUrl); var webhookMessage = new WebhookMessage(WebhookId: webhookId, EventId: eventId, EventType: eventType, Event: webhookEvent, InstanceId: instanceId, InstanceName: _context.Creds.GetInstanceName(), CreatedAt: eventData.CreatedAt, SasUrl: eventData.SasUrl);
var opts = await _context.FeatureManagerSnapshot.IsEnabledAsync(FeatureFlagConstants.EnableSlimEventSerialization) switch {
true => _optionsSlim,
false => _options
};
if (messageFormat != null && messageFormat == WebhookMessageFormat.EventGrid) { if (messageFormat != null && messageFormat == WebhookMessageFormat.EventGrid) {
var eventGridMessage = new[] { new WebhookMessageEventGrid(Id: eventId, Data: webhookMessage, DataVersion: "2.0.0", Subject: _context.Creds.GetInstanceName(), EventType: eventType, EventTime: DateTimeOffset.UtcNow) }; var eventGridMessage = new[] { new WebhookMessageEventGrid(Id: eventId, Data: webhookMessage, DataVersion: "2.0.0", Subject: _context.Creds.GetInstanceName(), EventType: eventType, EventTime: DateTimeOffset.UtcNow) };
data = JsonSerializer.Serialize(eventGridMessage, options: EntityConverter.GetJsonSerializerOptions()); data = JsonSerializer.Serialize(eventGridMessage, options: opts);
} else { } else {
data = JsonSerializer.Serialize(webhookMessage, options: EntityConverter.GetJsonSerializerOptions()); data = JsonSerializer.Serialize(webhookMessage, options: opts);
} }
string? digest = null; string? digest = null;

View File

@ -0,0 +1,76 @@
using System.Collections;
using System.Reflection;
using System.Text.Json;
using System.Text.Json.Serialization;
namespace Microsoft.OneFuzz.Service {
public class RemoveUserInfo : JsonConverter<UserInfo> {
public override UserInfo? Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options) {
throw new NotSupportedException("reading UserInfo is not supported");
}
public override void Write(Utf8JsonWriter writer, UserInfo value, JsonSerializerOptions options) {
writer.WriteStartObject();
writer.WriteEndObject();
}
}
/// <summary>
/// <b>THIS IS A WRITE ONLY JSON CONVERTER</b>
/// <br/>
/// It should only be used when serializing events to be sent outside of the service
/// </summary>
public class EventExportConverter<T> : JsonConverter<T>
where T : DownloadableEventMessage {
public override T? Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options) {
throw new NotSupportedException("This converter should only be used when serializing event messages to sent outside of the service");
}
public override void Write(Utf8JsonWriter writer, T value, JsonSerializerOptions options) {
BoundedSerializer.WriteInternal(writer, value, options);
}
}
public class BoundedSerializer {
private static HashSet<Type> boundedTypes = new HashSet<Type>{
typeof(Guid),
typeof(DateTime),
typeof(int),
typeof(bool),
typeof(float),
typeof(double),
typeof(long),
typeof(char),
typeof(Uri)
};
public static void WriteInternal(Utf8JsonWriter writer, object type, JsonSerializerOptions options) {
writer.WriteStartObject();
var properties = type.GetType().GetProperties();
foreach (var property in properties) {
if (property.GetValue(type, null) == null
|| typeof(IEnumerable).IsAssignableFrom(property.PropertyType)
|| type.GetType() == property.PropertyType) {
continue;
}
if (HasBoundedSerialization(property)) {
var serialized = JsonSerializer.Serialize(property.GetValue(type, null), property.PropertyType, options);
if (!string.IsNullOrEmpty(serialized)) {
writer.WritePropertyName(property.Name);
writer.WriteRawValue(serialized);
}
} else if (property.PropertyType.IsClass) {
writer.WritePropertyName(property.Name);
WriteInternal(writer, property.GetValue(type, null)!, options);
}
}
writer.WriteEndObject();
}
public static bool HasBoundedSerialization(PropertyInfo propertyInfo) {
return propertyInfo.PropertyType.IsEnum ||
boundedTypes.Contains(propertyInfo.PropertyType) ||
typeof(IValidatedString).IsAssignableFrom(propertyInfo.PropertyType);
}
}
}

View File

@ -0,0 +1,219 @@
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text.Json;
using FluentAssertions;
using FsCheck;
using FsCheck.Xunit;
using Microsoft.OneFuzz.Service;
using Microsoft.OneFuzz.Service.OneFuzzLib.Orm;
using Xunit;
namespace Tests;
public class EventExportConverterTests {
enum Color {
Red,
Blue
}
[Fact]
public void BaseTypesAreBounded() {
var a = new {
guid = Guid.NewGuid(),
date = new DateTime(),
en = Color.Red,
b = 1,
boo = false,
flo = float.Pi,
doub = double.Tau,
lon = long.MinValue,
cha = 'a'
};
a.GetType().GetProperties().All(p => BoundedSerializer.HasBoundedSerialization(p)).Should().BeTrue();
}
[Fact]
public void StringIsNotBounded() {
var a = new {
bad = "this is not bounded"
};
BoundedSerializer.HasBoundedSerialization(a.GetType().GetProperty("bad")!).Should().BeFalse();
}
[Fact]
public void ValidatedStringIsBounded() {
var a = new {
scalesetid = ScalesetId.Parse("abc-123")
};
BoundedSerializer.HasBoundedSerialization(a.GetType().GetProperty("scalesetid")!).Should().BeTrue();
}
[Fact]
public void ComplexObjectsAreSerialized() {
var randomGuid = Guid.NewGuid();
var a = new DownloadableEventMessage(
randomGuid,
EventType.CrashReported,
new EventCrashReported(
new Report(
"https://example.com",
null,
"target.exe",
"crash",
string.Empty,
new List<string> { "this", "is", "a", "stacktrace" },
string.Empty,
string.Empty,
null,
Guid.NewGuid(),
Guid.NewGuid(),
null,
null,
null,
null,
null,
null,
null,
null,
null,
null,
null,
null
),
Container.Parse("this-is-a-container"),
"crash-abc123",
null
),
Guid.NewGuid(),
"onefuzz",
DateTime.Now,
new Uri("https://example.com"),
null
);
var serializerOptions = new JsonSerializerOptions(EntityConverter.GetJsonSerializerOptions());
serializerOptions.Converters.Add(new EventExportConverter<DownloadableEventMessage>());
var serialized = JsonSerializer.Serialize(a, serializerOptions);
serialized.Should().NotBeNullOrEmpty();
serialized.Should().NotContain("stacktrace"); // List<string> is not serialized
serialized.Should().NotContain("crash-abc123"); // string is not serialized
serialized.Should().Contain("this-is-a-container"); // ValidatedString is serialized
serialized.Should().Contain("crash_reported"); // Enum is serialized
serialized.Should().Contain(DateTime.Now.Year.ToString()); // DateTime is serialized
serialized.Should().Contain(randomGuid.ToString()); // Guid id serialized
}
[Fact]
public void TestWebhookMessage() {
var a = new WebhookMessageEventGrid(
"2.0.0",
"eventsubject",
EventType.JobCreated,
DateTime.Now,
Guid.NewGuid(),
new WebhookMessage(
Guid.NewGuid(),
EventType.JobCreated,
new EventJobCreated(
Guid.NewGuid(),
new JobConfig("some project", "some name", "some build", 1, "some logs"),
null,
"8.0"),
Guid.NewGuid(),
"onefuzz",
Guid.NewGuid(),
DateTime.Now,
new Uri("https://example.com")
)
);
var serializerOptions = new JsonSerializerOptions(EntityConverter.GetJsonSerializerOptions());
serializerOptions.Converters.Add(new EventExportConverter<WebhookMessage>());
var serialized = JsonSerializer.Serialize(a, serializerOptions);
serialized.Should().Contain("eventsubject");
serialized.Should().NotContain("some project");
}
public class EventExportConverterSerializationTests {
private readonly JsonSerializerOptions _opts = new JsonSerializerOptions(EntityConverter.GetJsonSerializerOptions());
public EventExportConverterSerializationTests() {
_ = Arb.Register<Arbitraries>();
_opts.Converters.Add(new EventExportConverter<DownloadableEventMessage>());
}
void Test<T>(T v) {
// TODO: Try cloning/creating a new serializer options from the existing one?
var serialized = JsonSerializer.Serialize(v, _opts);
var _ = JsonSerializer.Deserialize<dynamic>(serialized);
}
[Property]
public void EventNodeHeartbeat(EventNodeHeartbeat e) => Test(e);
[Property]
public void EventTaskHeartbeat(EventTaskHeartbeat e) => Test(e);
[Property]
public void EventTaskStopped(EventTaskStopped e) => Test(e);
[Property]
public void EventInstanceConfigUpdated(EventInstanceConfigUpdated e) => Test(e);
[Property]
public void EventProxyCreated(EventProxyCreated e) => Test(e);
[Property]
public void EventProxyDeleted(EventProxyDeleted e) => Test(e);
[Property]
public void EventProxyFailed(EventProxyFailed e) => Test(e);
[Property]
public void EventProxyStateUpdated(EventProxyStateUpdated e) => Test(e);
[Property]
public void EventCrashReported(EventCrashReported e) => Test(e);
[Property]
public void EventRegressionReported(EventRegressionReported e) => Test(e);
[Property]
public void EventFileAdded(EventFileAdded e) => Test(e);
[Property]
public void EventTaskFailed(EventTaskFailed e) => Test(e);
[Property]
public void EventTaskStateUpdated(EventTaskStateUpdated e) => Test(e);
[Property]
public void EventScalesetFailed(EventScalesetFailed e) => Test(e);
[Property]
public void EventScalesetResizeScheduled(EventScalesetResizeScheduled e) => Test(e);
[Property]
public void EventScalesetStateUpdated(EventScalesetStateUpdated e) => Test(e);
[Property]
public void EventNodeDeleted(EventNodeDeleted e) => Test(e);
[Property]
public void EventNodeCreated(EventNodeCreated e) => Test(e);
[Property]
public void EventMessage(DownloadableEventMessage e) => Test(e);
}
}

View File

@ -530,12 +530,13 @@ namespace Tests {
//Sample function on how repro a failing test run, using Replay //Sample function on how repro a failing test run, using Replay
//functionality of FsCheck. Feel free to //functionality of FsCheck. Feel free to
/*
[Fact]
void Replay() { void Replay() {
var seed = FsCheck.Random.StdGen.NewStdGen(811038773, 297085737); var seed = FsCheck.Random.StdGen.NewStdGen(1687595065, 297240661);
var p = Prop.ForAll((NotificationTemplate x) => NotificationTemplate(x)); var p = Prop.ForAll((DownloadableEventMessage x) => EventMessage(x));
p.Check(new Configuration { Replay = seed }); p.Check(new Configuration { Replay = seed });
} }
*/
} }
} }

View File

@ -102,4 +102,17 @@ resource enableContainerRetentionPolicies 'Microsoft.AppConfiguration/configurat
} }
} }
resource enableSlimEventSerialization 'Microsoft.AppConfiguration/configurationStores/keyValues@2021-10-01-preview' = {
parent: featureFlags
name: '.appconfig.featureflag~2FEnableSlimEventSerialization'
properties: {
value: string({
id: 'EnableSlimEventSerialization'
description: 'Enable serializing events as smaller payloads'
enabled: false
})
contentType: 'application/vnd.microsoft.appconfig.ff+json;charset=utf-8'
}
}
output AppConfigEndpoint string = 'https://${appConfigName}.azconfig.io' output AppConfigEndpoint string = 'https://${appConfigName}.azconfig.io'