Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions src/Microsoft.OpenApi/Models/OpenApiConstants.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -480,6 +480,11 @@ public static class OpenApiConstants
/// </summary>
public const string Properties = "properties";

/// <summary>
/// Field: UnrecognizedKeywords
/// </summary>
public const string UnrecognizedKeywords = "unrecognizedKeywords";

/// <summary>
/// Field: Pattern Properties
/// </summary>
Expand Down
73 changes: 28 additions & 45 deletions src/Microsoft.OpenApi/Models/OpenApiSchema.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -15,12 +15,8 @@ namespace Microsoft.OpenApi.Models
/// <summary>
/// The Schema Object allows the definition of input and output data types.
/// </summary>
public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiReferenceable, IOpenApiSerializable
public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiReferenceable
{
private JsonNode _example;
private JsonNode _default;
private IList<JsonNode> _examples;

/// <summary>
/// Follow JSON Schema definition. Short text providing information about the data.
/// </summary>
Expand DownExpand Up@@ -148,11 +144,7 @@ public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiRe
/// Unlike JSON Schema, the value MUST conform to the defined type for the Schema Object defined at the same level.
/// For example, if type is string, then default can be "foo" but cannot be 1.
/// </summary>
public virtual JsonNode Default
{
get => _default;
set => _default = value;
}
public virtual JsonNode Default { get; set; }

/// <summary>
/// Relevant only for Schema "properties" definitions. Declares the property as "read only".
Expand DownExpand Up@@ -273,22 +265,14 @@ public virtual JsonNode Default
/// To represent examples that cannot be naturally represented in JSON or YAML,
/// a string value can be used to contain the example with escaping where necessary.
/// </summary>
public virtual JsonNode Example
{
get => _example;
set => _example = value;
}
public virtual JsonNode Example { get; set; }

/// <summary>
/// A free-form property to include examples of an instance for this schema.
/// To represent examples that cannot be naturally represented in JSON or YAML,
/// a list of values can be used to contain the examples with escaping where necessary.
/// </summary>
public virtual IList<JsonNode> Examples
{
get => _examples;
set => _examples = value;
}
public virtual IList<JsonNode> Examples { get; set; }

/// <summary>
/// Follow JSON Schema definition: https://tools.ietf.org/html/draft-fge-json-schema-validation-00
Expand DownExpand Up@@ -327,6 +311,11 @@ public virtual IList<JsonNode> Examples
/// </summary>
public virtual IDictionary<string, IOpenApiExtension> Extensions { get; set; } = new Dictionary<string, IOpenApiExtension>();

/// <summary>
/// This object stores any unrecognized keywords found in the schema.
/// </summary>
public virtual IDictionary<string, JsonNode> UnrecognizedKeywords { get; set; } = new Dictionary<string, JsonNode>();

/// <summary>
/// Indicates object is a placeholder reference to an actual object and does not contain valid data.
/// </summary>
Expand DownExpand Up@@ -373,7 +362,7 @@ public OpenApiSchema(OpenApiSchema schema)
MinLength = schema?.MinLength ?? MinLength;
Pattern = schema?.Pattern ?? Pattern;
MultipleOf = schema?.MultipleOf ?? MultipleOf;
_default = schema?.Default != null ? JsonNodeCloneHelper.Clone(schema?.Default) : null;
Default = schema?.Default != null ? JsonNodeCloneHelper.Clone(schema?.Default) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
ReadOnly = schema?.ReadOnly ?? ReadOnly;
WriteOnly = schema?.WriteOnly ?? WriteOnly;
AllOf = schema?.AllOf != null ? new List<OpenApiSchema>(schema.AllOf) : null;
Expand All@@ -392,8 +381,8 @@ public OpenApiSchema(OpenApiSchema schema)
AdditionalPropertiesAllowed = schema?.AdditionalPropertiesAllowed ?? AdditionalPropertiesAllowed;
AdditionalProperties = schema?.AdditionalProperties != null ? new(schema?.AdditionalProperties) : null;
Discriminator = schema?.Discriminator != null ? new(schema?.Discriminator) : null;
_example = schema?.Example != null ? JsonNodeCloneHelper.Clone(schema?.Example) : null;
_examples = schema?.Examples != null ? new List<JsonNode>(schema.Examples) : null;
Example = schema?.Example != null ? JsonNodeCloneHelper.Clone(schema?.Example) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
Examples = schema?.Examples != null ? new List<JsonNode>(schema.Examples) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
Enum = schema?.Enum != null ? new List<JsonNode>(schema.Enum) : null;
Nullable = schema?.Nullable ?? Nullable;
ExternalDocs = schema?.ExternalDocs != null ? new(schema?.ExternalDocs) : null;
Expand All@@ -403,6 +392,7 @@ public OpenApiSchema(OpenApiSchema schema)
UnresolvedReference = schema?.UnresolvedReference ?? UnresolvedReference;
Reference = schema?.Reference != null ? new(schema?.Reference) : null;
Annotations = schema?.Annotations != null ? new Dictionary<string, object>(schema?.Annotations) : null;
UnrecognizedKeywords = schema?.UnrecognizedKeywords != null ? new Dictionary<string, JsonNode>(schema?.UnrecognizedKeywords) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
}

/// <summary>
Expand DownExpand Up@@ -430,7 +420,7 @@ public void SerializeInternal(IOpenApiWriter writer, OpenApiSpecVersion version,

if (version == OpenApiSpecVersion.OpenApi3_1)
{
WriteV31Properties(writer);
WriteJsonSchemaKeywords(writer);
}

// title
Expand DownExpand Up@@ -554,6 +544,12 @@ public void SerializeInternal(IOpenApiWriter writer, OpenApiSpecVersion version,
// extensions
writer.WriteExtensions(Extensions, version);

// Unrecognized keywords
if (UnrecognizedKeywords.Any())
{
writer.WriteOptionalMap(OpenApiConstants.UnrecognizedKeywords, UnrecognizedKeywords, (w,s) => w.WriteAny(s));
}

writer.WriteEndObject();
}

Expand All@@ -564,7 +560,7 @@ public virtual void SerializeAsV2(IOpenApiWriter writer)
SerializeAsV2(writer: writer, parentRequiredProperties: new HashSet<string>(), propertyName: null);
}

internal void WriteV31Properties(IOpenApiWriter writer)
internal void WriteJsonSchemaKeywords(IOpenApiWriter writer)
{
writer.WriteProperty(OpenApiConstants.Id, Id);
writer.WriteProperty(OpenApiConstants.DollarSchema, Schema);
Expand All@@ -577,7 +573,7 @@ internal void WriteV31Properties(IOpenApiWriter writer)
writer.WriteProperty(OpenApiConstants.V31ExclusiveMaximum, V31ExclusiveMaximum);
writer.WriteProperty(OpenApiConstants.V31ExclusiveMinimum, V31ExclusiveMinimum);
writer.WriteProperty(OpenApiConstants.UnevaluatedProperties, UnevaluatedProperties, false);
writer.WriteOptionalCollection(OpenApiConstants.Examples, _examples, (nodeWriter, s) => nodeWriter.WriteAny(s));
writer.WriteOptionalCollection(OpenApiConstants.Examples, Examples, (nodeWriter, s) => nodeWriter.WriteAny(s));
writer.WriteOptionalMap(OpenApiConstants.PatternProperties, PatternProperties, (w, s) => s.SerializeAsV31(w));
}

Expand DownExpand Up@@ -820,15 +816,9 @@ private void SerializeTypeProperty(JsonSchemaType? type, IOpenApiWriter writer,
}
else
{
var list = new List<JsonSchemaType>();
foreach (JsonSchemaType flag in jsonSchemaTypeValues)
{
if (type.Value.HasFlag(flag))
{
list.Add(flag);
}
}

var list = (from JsonSchemaType flag in jsonSchemaTypeValues
where type.Value.HasFlag(flag)
select flag).ToList();
writer.WriteOptionalCollection(OpenApiConstants.Type, list, (w, s) => w.WriteValue(s.ToIdentifier()));
}
}
Expand All@@ -850,16 +840,9 @@ private void UpCastSchemaTypeToV31(JsonSchemaType? type, IOpenApiWriter writer)
{
// create a new array and insert the type and "null" as values
Type = type | JsonSchemaType.Null;
var list = new List<string>();
foreach (JsonSchemaType? flag in jsonSchemaTypeValues)
{
// Check if the flag is set in 'type' using a bitwise AND operation
if (Type.Value.HasFlag(flag))
{
list.Add(flag.ToIdentifier());
}
}

var list = (from JsonSchemaType? flag in jsonSchemaTypeValues// Check if the flag is set in 'type' using a bitwise AND operation
where Type.Value.HasFlag(flag)
select flag.ToIdentifier()).ToList();
writer.WriteOptionalCollection(OpenApiConstants.Type, list, (w, s) => w.WriteValue(s));
}

Expand Down
14 changes: 13 additions & 1 deletion src/Microsoft.OpenApi/Reader/V31/OpenApiSchemaDeserializer.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -7,6 +7,8 @@
using Microsoft.OpenApi.Reader.ParseNodes;
using System.Collections.Generic;
using System.Globalization;
using System.Linq;
using System.Text.Json.Nodes;

namespace Microsoft.OpenApi.Reader.V31
{
Expand DownExpand Up@@ -254,7 +256,17 @@ public static OpenApiSchema LoadSchema(ParseNode node, OpenApiDocument hostDocum

foreach (var propertyNode in mapNode)
{
propertyNode.ParseField(schema, _openApiSchemaFixedFields, _openApiSchemaPatternFields);
bool isRecognized = _openApiSchemaFixedFields.ContainsKey(propertyNode.Name) ||
_openApiSchemaPatternFields.Any(p => p.Key(propertyNode.Name));
Comment thread
baywet marked this conversation as resolved.

if (isRecognized)
{
propertyNode.ParseField(schema, _openApiSchemaFixedFields, _openApiSchemaPatternFields);
}
else
{
schema.UnrecognizedKeywords[propertyNode.Name] = propertyNode.JsonNode;
}
}

if (schema.Extensions.ContainsKey(OpenApiConstants.NullableExtension))
Expand Down
20 changes: 20 additions & 0 deletions src/Microsoft.OpenApi/Writers/OpenApiWriterExtensions.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -5,6 +5,7 @@
using System.Collections;
using System.Collections.Generic;
using System.Linq;
using System.Text.Json.Nodes;
using Microsoft.OpenApi.Interfaces;

namespace Microsoft.OpenApi.Writers
Expand DownExpand Up@@ -253,6 +254,25 @@ public static void WriteRequiredMap(
writer.WriteMapInternal(name, elements, action);
}

/// <summary>
/// Write the optional Open API element map (string to string mapping).
/// </summary>
/// <param name="writer">The Open API writer.</param>
/// <param name="name">The property name.</param>
/// <param name="elements">The map values.</param>
/// <param name="action">The map element writer action.</param>
public static void WriteOptionalMap(
this IOpenApiWriter writer,
string name,
IDictionary<string, JsonNode> elements,
Action<IOpenApiWriter, JsonNode> action)
{
if (elements != null && elements.Any())
{
writer.WriteMapInternal(name, elements, action);
}
}

/// <summary>
/// Write the optional Open API element map (string to string mapping).
/// </summary>
Expand Down
Original file line numberDiff line numberDiff line change
Expand Up@@ -495,5 +495,21 @@ public void ParseSchemaWithConstWorks()
var schemaString = writer.ToString();
schemaString.MakeLineBreaksEnvironmentNeutral().Should().Be(expected.MakeLineBreaksEnvironmentNeutral());
}

[Fact]
public void ParseSchemaWithUnrecognizedKeywordsWorks()
{
var input = @"{
""type"": ""string"",
""format"": ""date-time"",
""customKeyword"": ""customValue"",
""anotherKeyword"": 42,
""x-test"": ""test""
}
";
var schema = OpenApiModelFactory.Parse<OpenApiSchema>(input, OpenApiSpecVersion.OpenApi3_1, out _, "json");
schema.UnrecognizedKeywords.Should().HaveCount(2);
}

}
}
Original file line numberDiff line numberDiff line change
Expand Up@@ -28,14 +28,3 @@ required:
- name

$dynamicAnchor: "addressDef"
definitions:
address:
$dynamicAnchor: "addressDef"
type: "object"
properties:
street:
type: "string"
city:
type: "string"
postalCode:
type: "string"
41 changes: 34 additions & 7 deletions test/Microsoft.OpenApi.Tests/Models/OpenApiSchemaTests.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -22,7 +22,7 @@ namespace Microsoft.OpenApi.Tests.Models
[Collection("DefaultSettings")]
public class OpenApiSchemaTests
{
public static OpenApiSchema BasicSchema = new();
private static readonly OpenApiSchema BasicSchema = new();

public static readonly OpenApiSchema AdvancedSchemaNumber = new()
{
Expand DownExpand Up@@ -602,15 +602,42 @@ public void OpenApiWalkerVisitsOpenApiSchemaNot()
// Assert
visitor.Titles.Count.Should().Be(2);
}
}

internal class SchemaVisitor : OpenApiVisitorBase
{
public List<string> Titles = new();
[Fact]
public void SerializeSchemaWithUnrecognizedPropertiesWorks()
{
// Arrange
var schema = new OpenApiSchema
{
UnrecognizedKeywords = new Dictionary<string, JsonNode>()
{
["customKeyWord"] = "bar",
["anotherKeyword"] = 42
}
};

public override void Visit(OpenApiSchema schema)
var expected = @"{
""unrecognizedKeywords"": {
""customKeyWord"": ""bar"",
""anotherKeyword"": 42
}
}";

// Act
var actual = schema.SerializeAsJson(OpenApiSpecVersion.OpenApi3_1);

// Assert
actual.MakeLineBreaksEnvironmentNeutral().Should().Be(expected.MakeLineBreaksEnvironmentNeutral());
}

internal class SchemaVisitor : OpenApiVisitorBase
{
Titles.Add(schema.Title);
public List<string> Titles = new();

public override void Visit(OpenApiSchema schema)
{
Titles.Add(schema.Title);
}
}
}
}
Original file line numberDiff line numberDiff line change
Expand Up@@ -512,6 +512,7 @@ namespace Microsoft.OpenApi.Models
public const string Type = "type";
public const string UnevaluatedProperties = "unevaluatedProperties";
public const string UniqueItems = "uniqueItems";
public const string UnrecognizedKeywords = "unrecognizedKeywords";
public const string Url = "url";
public const string V2ReferenceUri = "https://registry/definitions/";
public const string V31ExclusiveMaximum = "exclusiveMaximum";
Expand DownExpand Up@@ -925,6 +926,7 @@ namespace Microsoft.OpenApi.Models
public virtual bool UnEvaluatedProperties { get; set; }
public virtual bool UnevaluatedProperties { get; set; }
public virtual bool? UniqueItems { get; set; }
public virtual System.Collections.Generic.IDictionary<string, System.Text.Json.Nodes.JsonNode> UnrecognizedKeywords { get; set; }
public virtual bool UnresolvedReference { get; set; }
public virtual decimal? V31ExclusiveMaximum { get; set; }
public virtual decimal? V31ExclusiveMinimum { get; set; }
Expand DownExpand Up@@ -1859,6 +1861,7 @@ namespace Microsoft.OpenApi.Writers
public static void WriteOptionalCollection<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IEnumerable<T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, T> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, bool> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, bool> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, string> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, string> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, System.Text.Json.Nodes.JsonNode> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, System.Text.Json.Nodes.JsonNode> action) { }
public static void WriteOptionalMap<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, T> action)
where T : Microsoft.OpenApi.Interfaces.IOpenApiElement { }
public static void WriteOptionalMap<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, string, T> action)
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions src/Microsoft.OpenApi/Models/OpenApiConstants.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -480,6 +480,11 @@ public static class OpenApiConstants
/// </summary>
public const string Properties = "properties";

/// <summary>
/// Field: UnrecognizedKeywords
/// </summary>
public const string UnrecognizedKeywords = "unrecognizedKeywords";

/// <summary>
/// Field: Pattern Properties
/// </summary>
Expand Down
73 changes: 28 additions & 45 deletions src/Microsoft.OpenApi/Models/OpenApiSchema.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -15,12 +15,8 @@ namespace Microsoft.OpenApi.Models
/// <summary>
/// The Schema Object allows the definition of input and output data types.
/// </summary>
public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiReferenceable, IOpenApiSerializable
public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiReferenceable
{
private JsonNode _example;
private JsonNode _default;
private IList<JsonNode> _examples;

/// <summary>
/// Follow JSON Schema definition. Short text providing information about the data.
/// </summary>
Expand DownExpand Up@@ -148,11 +144,7 @@ public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiRe
/// Unlike JSON Schema, the value MUST conform to the defined type for the Schema Object defined at the same level.
/// For example, if type is string, then default can be "foo" but cannot be 1.
/// </summary>
public virtual JsonNode Default
{
get => _default;
set => _default = value;
}
public virtual JsonNode Default { get; set; }

/// <summary>
/// Relevant only for Schema "properties" definitions. Declares the property as "read only".
Expand DownExpand Up@@ -273,22 +265,14 @@ public virtual JsonNode Default
/// To represent examples that cannot be naturally represented in JSON or YAML,
/// a string value can be used to contain the example with escaping where necessary.
/// </summary>
public virtual JsonNode Example
{
get => _example;
set => _example = value;
}
public virtual JsonNode Example { get; set; }

/// <summary>
/// A free-form property to include examples of an instance for this schema.
/// To represent examples that cannot be naturally represented in JSON or YAML,
/// a list of values can be used to contain the examples with escaping where necessary.
/// </summary>
public virtual IList<JsonNode> Examples
{
get => _examples;
set => _examples = value;
}
public virtual IList<JsonNode> Examples { get; set; }

/// <summary>
/// Follow JSON Schema definition: https://tools.ietf.org/html/draft-fge-json-schema-validation-00
Expand DownExpand Up@@ -327,6 +311,11 @@ public virtual IList<JsonNode> Examples
/// </summary>
public virtual IDictionary<string, IOpenApiExtension> Extensions { get; set; } = new Dictionary<string, IOpenApiExtension>();

/// <summary>
/// This object stores any unrecognized keywords found in the schema.
/// </summary>
public virtual IDictionary<string, JsonNode> UnrecognizedKeywords { get; set; } = new Dictionary<string, JsonNode>();

/// <summary>
/// Indicates object is a placeholder reference to an actual object and does not contain valid data.
/// </summary>
Expand DownExpand Up@@ -373,7 +362,7 @@ public OpenApiSchema(OpenApiSchema schema)
MinLength = schema?.MinLength ?? MinLength;
Pattern = schema?.Pattern ?? Pattern;
MultipleOf = schema?.MultipleOf ?? MultipleOf;
_default = schema?.Default != null ? JsonNodeCloneHelper.Clone(schema?.Default) : null;
Default = schema?.Default != null ? JsonNodeCloneHelper.Clone(schema?.Default) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
ReadOnly = schema?.ReadOnly ?? ReadOnly;
WriteOnly = schema?.WriteOnly ?? WriteOnly;
AllOf = schema?.AllOf != null ? new List<OpenApiSchema>(schema.AllOf) : null;
Expand All@@ -392,8 +381,8 @@ public OpenApiSchema(OpenApiSchema schema)
AdditionalPropertiesAllowed = schema?.AdditionalPropertiesAllowed ?? AdditionalPropertiesAllowed;
AdditionalProperties = schema?.AdditionalProperties != null ? new(schema?.AdditionalProperties) : null;
Discriminator = schema?.Discriminator != null ? new(schema?.Discriminator) : null;
_example = schema?.Example != null ? JsonNodeCloneHelper.Clone(schema?.Example) : null;
_examples = schema?.Examples != null ? new List<JsonNode>(schema.Examples) : null;
Example = schema?.Example != null ? JsonNodeCloneHelper.Clone(schema?.Example) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
Examples = schema?.Examples != null ? new List<JsonNode>(schema.Examples) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
Enum = schema?.Enum != null ? new List<JsonNode>(schema.Enum) : null;
Nullable = schema?.Nullable ?? Nullable;
ExternalDocs = schema?.ExternalDocs != null ? new(schema?.ExternalDocs) : null;
Expand All@@ -403,6 +392,7 @@ public OpenApiSchema(OpenApiSchema schema)
UnresolvedReference = schema?.UnresolvedReference ?? UnresolvedReference;
Reference = schema?.Reference != null ? new(schema?.Reference) : null;
Annotations = schema?.Annotations != null ? new Dictionary<string, object>(schema?.Annotations) : null;
UnrecognizedKeywords = schema?.UnrecognizedKeywords != null ? new Dictionary<string, JsonNode>(schema?.UnrecognizedKeywords) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
}

/// <summary>
Expand DownExpand Up@@ -430,7 +420,7 @@ public void SerializeInternal(IOpenApiWriter writer, OpenApiSpecVersion version,

if (version == OpenApiSpecVersion.OpenApi3_1)
{
WriteV31Properties(writer);
WriteJsonSchemaKeywords(writer);
}

// title
Expand DownExpand Up@@ -554,6 +544,12 @@ public void SerializeInternal(IOpenApiWriter writer, OpenApiSpecVersion version,
// extensions
writer.WriteExtensions(Extensions, version);

// Unrecognized keywords
if (UnrecognizedKeywords.Any())
{
writer.WriteOptionalMap(OpenApiConstants.UnrecognizedKeywords, UnrecognizedKeywords, (w,s) => w.WriteAny(s));
}

writer.WriteEndObject();
}

Expand All@@ -564,7 +560,7 @@ public virtual void SerializeAsV2(IOpenApiWriter writer)
SerializeAsV2(writer: writer, parentRequiredProperties: new HashSet<string>(), propertyName: null);
}

internal void WriteV31Properties(IOpenApiWriter writer)
internal void WriteJsonSchemaKeywords(IOpenApiWriter writer)
{
writer.WriteProperty(OpenApiConstants.Id, Id);
writer.WriteProperty(OpenApiConstants.DollarSchema, Schema);
Expand All@@ -577,7 +573,7 @@ internal void WriteV31Properties(IOpenApiWriter writer)
writer.WriteProperty(OpenApiConstants.V31ExclusiveMaximum, V31ExclusiveMaximum);
writer.WriteProperty(OpenApiConstants.V31ExclusiveMinimum, V31ExclusiveMinimum);
writer.WriteProperty(OpenApiConstants.UnevaluatedProperties, UnevaluatedProperties, false);
writer.WriteOptionalCollection(OpenApiConstants.Examples, _examples, (nodeWriter, s) => nodeWriter.WriteAny(s));
writer.WriteOptionalCollection(OpenApiConstants.Examples, Examples, (nodeWriter, s) => nodeWriter.WriteAny(s));
writer.WriteOptionalMap(OpenApiConstants.PatternProperties, PatternProperties, (w, s) => s.SerializeAsV31(w));
}

Expand DownExpand Up@@ -820,15 +816,9 @@ private void SerializeTypeProperty(JsonSchemaType? type, IOpenApiWriter writer,
}
else
{
var list = new List<JsonSchemaType>();
foreach (JsonSchemaType flag in jsonSchemaTypeValues)
{
if (type.Value.HasFlag(flag))
{
list.Add(flag);
}
}

var list = (from JsonSchemaType flag in jsonSchemaTypeValues
where type.Value.HasFlag(flag)
select flag).ToList();
writer.WriteOptionalCollection(OpenApiConstants.Type, list, (w, s) => w.WriteValue(s.ToIdentifier()));
}
}
Expand All@@ -850,16 +840,9 @@ private void UpCastSchemaTypeToV31(JsonSchemaType? type, IOpenApiWriter writer)
{
// create a new array and insert the type and "null" as values
Type = type | JsonSchemaType.Null;
var list = new List<string>();
foreach (JsonSchemaType? flag in jsonSchemaTypeValues)
{
// Check if the flag is set in 'type' using a bitwise AND operation
if (Type.Value.HasFlag(flag))
{
list.Add(flag.ToIdentifier());
}
}

var list = (from JsonSchemaType? flag in jsonSchemaTypeValues// Check if the flag is set in 'type' using a bitwise AND operation
where Type.Value.HasFlag(flag)
select flag.ToIdentifier()).ToList();
writer.WriteOptionalCollection(OpenApiConstants.Type, list, (w, s) => w.WriteValue(s));
}

Expand Down
14 changes: 13 additions & 1 deletion src/Microsoft.OpenApi/Reader/V31/OpenApiSchemaDeserializer.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -7,6 +7,8 @@
using Microsoft.OpenApi.Reader.ParseNodes;
using System.Collections.Generic;
using System.Globalization;
using System.Linq;
using System.Text.Json.Nodes;

namespace Microsoft.OpenApi.Reader.V31
{
Expand DownExpand Up@@ -254,7 +256,17 @@ public static OpenApiSchema LoadSchema(ParseNode node, OpenApiDocument hostDocum

foreach (var propertyNode in mapNode)
{
propertyNode.ParseField(schema, _openApiSchemaFixedFields, _openApiSchemaPatternFields);
bool isRecognized = _openApiSchemaFixedFields.ContainsKey(propertyNode.Name) ||
_openApiSchemaPatternFields.Any(p => p.Key(propertyNode.Name));
Comment thread
baywet marked this conversation as resolved.

if (isRecognized)
{
propertyNode.ParseField(schema, _openApiSchemaFixedFields, _openApiSchemaPatternFields);
}
else
{
schema.UnrecognizedKeywords[propertyNode.Name] = propertyNode.JsonNode;
}
}

if (schema.Extensions.ContainsKey(OpenApiConstants.NullableExtension))
Expand Down
20 changes: 20 additions & 0 deletions src/Microsoft.OpenApi/Writers/OpenApiWriterExtensions.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -5,6 +5,7 @@
using System.Collections;
using System.Collections.Generic;
using System.Linq;
using System.Text.Json.Nodes;
using Microsoft.OpenApi.Interfaces;

namespace Microsoft.OpenApi.Writers
Expand DownExpand Up@@ -253,6 +254,25 @@ public static void WriteRequiredMap(
writer.WriteMapInternal(name, elements, action);
}

/// <summary>
/// Write the optional Open API element map (string to string mapping).
/// </summary>
/// <param name="writer">The Open API writer.</param>
/// <param name="name">The property name.</param>
/// <param name="elements">The map values.</param>
/// <param name="action">The map element writer action.</param>
public static void WriteOptionalMap(
this IOpenApiWriter writer,
string name,
IDictionary<string, JsonNode> elements,
Action<IOpenApiWriter, JsonNode> action)
{
if (elements != null && elements.Any())
{
writer.WriteMapInternal(name, elements, action);
}
}

/// <summary>
/// Write the optional Open API element map (string to string mapping).
/// </summary>
Expand Down
Original file line numberDiff line numberDiff line change
Expand Up@@ -495,5 +495,21 @@ public void ParseSchemaWithConstWorks()
var schemaString = writer.ToString();
schemaString.MakeLineBreaksEnvironmentNeutral().Should().Be(expected.MakeLineBreaksEnvironmentNeutral());
}

[Fact]
public void ParseSchemaWithUnrecognizedKeywordsWorks()
{
var input = @"{
""type"": ""string"",
""format"": ""date-time"",
""customKeyword"": ""customValue"",
""anotherKeyword"": 42,
""x-test"": ""test""
}
";
var schema = OpenApiModelFactory.Parse<OpenApiSchema>(input, OpenApiSpecVersion.OpenApi3_1, out _, "json");
schema.UnrecognizedKeywords.Should().HaveCount(2);
}

}
}
Original file line numberDiff line numberDiff line change
Expand Up@@ -28,14 +28,3 @@ required:
- name

$dynamicAnchor: "addressDef"
definitions:
address:
$dynamicAnchor: "addressDef"
type: "object"
properties:
street:
type: "string"
city:
type: "string"
postalCode:
type: "string"
41 changes: 34 additions & 7 deletions test/Microsoft.OpenApi.Tests/Models/OpenApiSchemaTests.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -22,7 +22,7 @@ namespace Microsoft.OpenApi.Tests.Models
[Collection("DefaultSettings")]
public class OpenApiSchemaTests
{
public static OpenApiSchema BasicSchema = new();
private static readonly OpenApiSchema BasicSchema = new();

public static readonly OpenApiSchema AdvancedSchemaNumber = new()
{
Expand DownExpand Up@@ -602,15 +602,42 @@ public void OpenApiWalkerVisitsOpenApiSchemaNot()
// Assert
visitor.Titles.Count.Should().Be(2);
}
}

internal class SchemaVisitor : OpenApiVisitorBase
{
public List<string> Titles = new();
[Fact]
public void SerializeSchemaWithUnrecognizedPropertiesWorks()
{
// Arrange
var schema = new OpenApiSchema
{
UnrecognizedKeywords = new Dictionary<string, JsonNode>()
{
["customKeyWord"] = "bar",
["anotherKeyword"] = 42
}
};

public override void Visit(OpenApiSchema schema)
var expected = @"{
""unrecognizedKeywords"": {
""customKeyWord"": ""bar"",
""anotherKeyword"": 42
}
}";

// Act
var actual = schema.SerializeAsJson(OpenApiSpecVersion.OpenApi3_1);

// Assert
actual.MakeLineBreaksEnvironmentNeutral().Should().Be(expected.MakeLineBreaksEnvironmentNeutral());
}

internal class SchemaVisitor : OpenApiVisitorBase
{
Titles.Add(schema.Title);
public List<string> Titles = new();

public override void Visit(OpenApiSchema schema)
{
Titles.Add(schema.Title);
}
}
}
}
Original file line numberDiff line numberDiff line change
Expand Up@@ -512,6 +512,7 @@ namespace Microsoft.OpenApi.Models
public const string Type = "type";
public const string UnevaluatedProperties = "unevaluatedProperties";
public const string UniqueItems = "uniqueItems";
public const string UnrecognizedKeywords = "unrecognizedKeywords";
public const string Url = "url";
public const string V2ReferenceUri = "https://registry/definitions/";
public const string V31ExclusiveMaximum = "exclusiveMaximum";
Expand DownExpand Up@@ -925,6 +926,7 @@ namespace Microsoft.OpenApi.Models
public virtual bool UnEvaluatedProperties { get; set; }
public virtual bool UnevaluatedProperties { get; set; }
public virtual bool? UniqueItems { get; set; }
public virtual System.Collections.Generic.IDictionary<string, System.Text.Json.Nodes.JsonNode> UnrecognizedKeywords { get; set; }
public virtual bool UnresolvedReference { get; set; }
public virtual decimal? V31ExclusiveMaximum { get; set; }
public virtual decimal? V31ExclusiveMinimum { get; set; }
Expand DownExpand Up@@ -1859,6 +1861,7 @@ namespace Microsoft.OpenApi.Writers
public static void WriteOptionalCollection<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IEnumerable<T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, T> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, bool> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, bool> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, string> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, string> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, System.Text.Json.Nodes.JsonNode> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, System.Text.Json.Nodes.JsonNode> action) { }
public static void WriteOptionalMap<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, T> action)
where T : Microsoft.OpenApi.Interfaces.IOpenApiElement { }
public static void WriteOptionalMap<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, string, T> action)
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions src/Microsoft.OpenApi/Models/OpenApiConstants.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -480,6 +480,11 @@ public static class OpenApiConstants
/// </summary>
public const string Properties = "properties";

/// <summary>
/// Field: UnrecognizedKeywords
/// </summary>
public const string UnrecognizedKeywords = "unrecognizedKeywords";

/// <summary>
/// Field: Pattern Properties
/// </summary>
Expand Down
73 changes: 28 additions & 45 deletions src/Microsoft.OpenApi/Models/OpenApiSchema.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -15,12 +15,8 @@ namespace Microsoft.OpenApi.Models
/// <summary>
/// The Schema Object allows the definition of input and output data types.
/// </summary>
public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiReferenceable, IOpenApiSerializable
public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiReferenceable
{
private JsonNode _example;
private JsonNode _default;
private IList<JsonNode> _examples;

/// <summary>
/// Follow JSON Schema definition. Short text providing information about the data.
/// </summary>
Expand DownExpand Up@@ -148,11 +144,7 @@ public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiRe
/// Unlike JSON Schema, the value MUST conform to the defined type for the Schema Object defined at the same level.
/// For example, if type is string, then default can be "foo" but cannot be 1.
/// </summary>
public virtual JsonNode Default
{
get => _default;
set => _default = value;
}
public virtual JsonNode Default { get; set; }

/// <summary>
/// Relevant only for Schema "properties" definitions. Declares the property as "read only".
Expand DownExpand Up@@ -273,22 +265,14 @@ public virtual JsonNode Default
/// To represent examples that cannot be naturally represented in JSON or YAML,
/// a string value can be used to contain the example with escaping where necessary.
/// </summary>
public virtual JsonNode Example
{
get => _example;
set => _example = value;
}
public virtual JsonNode Example { get; set; }

/// <summary>
/// A free-form property to include examples of an instance for this schema.
/// To represent examples that cannot be naturally represented in JSON or YAML,
/// a list of values can be used to contain the examples with escaping where necessary.
/// </summary>
public virtual IList<JsonNode> Examples
{
get => _examples;
set => _examples = value;
}
public virtual IList<JsonNode> Examples { get; set; }

/// <summary>
/// Follow JSON Schema definition: https://tools.ietf.org/html/draft-fge-json-schema-validation-00
Expand DownExpand Up@@ -327,6 +311,11 @@ public virtual IList<JsonNode> Examples
/// </summary>
public virtual IDictionary<string, IOpenApiExtension> Extensions { get; set; } = new Dictionary<string, IOpenApiExtension>();

/// <summary>
/// This object stores any unrecognized keywords found in the schema.
/// </summary>
public virtual IDictionary<string, JsonNode> UnrecognizedKeywords { get; set; } = new Dictionary<string, JsonNode>();

/// <summary>
/// Indicates object is a placeholder reference to an actual object and does not contain valid data.
/// </summary>
Expand DownExpand Up@@ -373,7 +362,7 @@ public OpenApiSchema(OpenApiSchema schema)
MinLength = schema?.MinLength ?? MinLength;
Pattern = schema?.Pattern ?? Pattern;
MultipleOf = schema?.MultipleOf ?? MultipleOf;
_default = schema?.Default != null ? JsonNodeCloneHelper.Clone(schema?.Default) : null;
Default = schema?.Default != null ? JsonNodeCloneHelper.Clone(schema?.Default) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
ReadOnly = schema?.ReadOnly ?? ReadOnly;
WriteOnly = schema?.WriteOnly ?? WriteOnly;
AllOf = schema?.AllOf != null ? new List<OpenApiSchema>(schema.AllOf) : null;
Expand All@@ -392,8 +381,8 @@ public OpenApiSchema(OpenApiSchema schema)
AdditionalPropertiesAllowed = schema?.AdditionalPropertiesAllowed ?? AdditionalPropertiesAllowed;
AdditionalProperties = schema?.AdditionalProperties != null ? new(schema?.AdditionalProperties) : null;
Discriminator = schema?.Discriminator != null ? new(schema?.Discriminator) : null;
_example = schema?.Example != null ? JsonNodeCloneHelper.Clone(schema?.Example) : null;
_examples = schema?.Examples != null ? new List<JsonNode>(schema.Examples) : null;
Example = schema?.Example != null ? JsonNodeCloneHelper.Clone(schema?.Example) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
Examples = schema?.Examples != null ? new List<JsonNode>(schema.Examples) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
Enum = schema?.Enum != null ? new List<JsonNode>(schema.Enum) : null;
Nullable = schema?.Nullable ?? Nullable;
ExternalDocs = schema?.ExternalDocs != null ? new(schema?.ExternalDocs) : null;
Expand All@@ -403,6 +392,7 @@ public OpenApiSchema(OpenApiSchema schema)
UnresolvedReference = schema?.UnresolvedReference ?? UnresolvedReference;
Reference = schema?.Reference != null ? new(schema?.Reference) : null;
Annotations = schema?.Annotations != null ? new Dictionary<string, object>(schema?.Annotations) : null;
UnrecognizedKeywords = schema?.UnrecognizedKeywords != null ? new Dictionary<string, JsonNode>(schema?.UnrecognizedKeywords) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
}

/// <summary>
Expand DownExpand Up@@ -430,7 +420,7 @@ public void SerializeInternal(IOpenApiWriter writer, OpenApiSpecVersion version,

if (version == OpenApiSpecVersion.OpenApi3_1)
{
WriteV31Properties(writer);
WriteJsonSchemaKeywords(writer);
}

// title
Expand DownExpand Up@@ -554,6 +544,12 @@ public void SerializeInternal(IOpenApiWriter writer, OpenApiSpecVersion version,
// extensions
writer.WriteExtensions(Extensions, version);

// Unrecognized keywords
if (UnrecognizedKeywords.Any())
{
writer.WriteOptionalMap(OpenApiConstants.UnrecognizedKeywords, UnrecognizedKeywords, (w,s) => w.WriteAny(s));
}

writer.WriteEndObject();
}

Expand All@@ -564,7 +560,7 @@ public virtual void SerializeAsV2(IOpenApiWriter writer)
SerializeAsV2(writer: writer, parentRequiredProperties: new HashSet<string>(), propertyName: null);
}

internal void WriteV31Properties(IOpenApiWriter writer)
internal void WriteJsonSchemaKeywords(IOpenApiWriter writer)
{
writer.WriteProperty(OpenApiConstants.Id, Id);
writer.WriteProperty(OpenApiConstants.DollarSchema, Schema);
Expand All@@ -577,7 +573,7 @@ internal void WriteV31Properties(IOpenApiWriter writer)
writer.WriteProperty(OpenApiConstants.V31ExclusiveMaximum, V31ExclusiveMaximum);
writer.WriteProperty(OpenApiConstants.V31ExclusiveMinimum, V31ExclusiveMinimum);
writer.WriteProperty(OpenApiConstants.UnevaluatedProperties, UnevaluatedProperties, false);
writer.WriteOptionalCollection(OpenApiConstants.Examples, _examples, (nodeWriter, s) => nodeWriter.WriteAny(s));
writer.WriteOptionalCollection(OpenApiConstants.Examples, Examples, (nodeWriter, s) => nodeWriter.WriteAny(s));
writer.WriteOptionalMap(OpenApiConstants.PatternProperties, PatternProperties, (w, s) => s.SerializeAsV31(w));
}

Expand DownExpand Up@@ -820,15 +816,9 @@ private void SerializeTypeProperty(JsonSchemaType? type, IOpenApiWriter writer,
}
else
{
var list = new List<JsonSchemaType>();
foreach (JsonSchemaType flag in jsonSchemaTypeValues)
{
if (type.Value.HasFlag(flag))
{
list.Add(flag);
}
}

var list = (from JsonSchemaType flag in jsonSchemaTypeValues
where type.Value.HasFlag(flag)
select flag).ToList();
writer.WriteOptionalCollection(OpenApiConstants.Type, list, (w, s) => w.WriteValue(s.ToIdentifier()));
}
}
Expand All@@ -850,16 +840,9 @@ private void UpCastSchemaTypeToV31(JsonSchemaType? type, IOpenApiWriter writer)
{
// create a new array and insert the type and "null" as values
Type = type | JsonSchemaType.Null;
var list = new List<string>();
foreach (JsonSchemaType? flag in jsonSchemaTypeValues)
{
// Check if the flag is set in 'type' using a bitwise AND operation
if (Type.Value.HasFlag(flag))
{
list.Add(flag.ToIdentifier());
}
}

var list = (from JsonSchemaType? flag in jsonSchemaTypeValues// Check if the flag is set in 'type' using a bitwise AND operation
where Type.Value.HasFlag(flag)
select flag.ToIdentifier()).ToList();
writer.WriteOptionalCollection(OpenApiConstants.Type, list, (w, s) => w.WriteValue(s));
}

Expand Down
14 changes: 13 additions & 1 deletion src/Microsoft.OpenApi/Reader/V31/OpenApiSchemaDeserializer.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -7,6 +7,8 @@
using Microsoft.OpenApi.Reader.ParseNodes;
using System.Collections.Generic;
using System.Globalization;
using System.Linq;
using System.Text.Json.Nodes;

namespace Microsoft.OpenApi.Reader.V31
{
Expand DownExpand Up@@ -254,7 +256,17 @@ public static OpenApiSchema LoadSchema(ParseNode node, OpenApiDocument hostDocum

foreach (var propertyNode in mapNode)
{
propertyNode.ParseField(schema, _openApiSchemaFixedFields, _openApiSchemaPatternFields);
bool isRecognized = _openApiSchemaFixedFields.ContainsKey(propertyNode.Name) ||
_openApiSchemaPatternFields.Any(p => p.Key(propertyNode.Name));
Comment thread
baywet marked this conversation as resolved.

if (isRecognized)
{
propertyNode.ParseField(schema, _openApiSchemaFixedFields, _openApiSchemaPatternFields);
}
else
{
schema.UnrecognizedKeywords[propertyNode.Name] = propertyNode.JsonNode;
}
}

if (schema.Extensions.ContainsKey(OpenApiConstants.NullableExtension))
Expand Down
20 changes: 20 additions & 0 deletions src/Microsoft.OpenApi/Writers/OpenApiWriterExtensions.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -5,6 +5,7 @@
using System.Collections;
using System.Collections.Generic;
using System.Linq;
using System.Text.Json.Nodes;
using Microsoft.OpenApi.Interfaces;

namespace Microsoft.OpenApi.Writers
Expand DownExpand Up@@ -253,6 +254,25 @@ public static void WriteRequiredMap(
writer.WriteMapInternal(name, elements, action);
}

/// <summary>
/// Write the optional Open API element map (string to string mapping).
/// </summary>
/// <param name="writer">The Open API writer.</param>
/// <param name="name">The property name.</param>
/// <param name="elements">The map values.</param>
/// <param name="action">The map element writer action.</param>
public static void WriteOptionalMap(
this IOpenApiWriter writer,
string name,
IDictionary<string, JsonNode> elements,
Action<IOpenApiWriter, JsonNode> action)
{
if (elements != null && elements.Any())
{
writer.WriteMapInternal(name, elements, action);
}
}

/// <summary>
/// Write the optional Open API element map (string to string mapping).
/// </summary>
Expand Down
Original file line numberDiff line numberDiff line change
Expand Up@@ -495,5 +495,21 @@ public void ParseSchemaWithConstWorks()
var schemaString = writer.ToString();
schemaString.MakeLineBreaksEnvironmentNeutral().Should().Be(expected.MakeLineBreaksEnvironmentNeutral());
}

[Fact]
public void ParseSchemaWithUnrecognizedKeywordsWorks()
{
var input = @"{
""type"": ""string"",
""format"": ""date-time"",
""customKeyword"": ""customValue"",
""anotherKeyword"": 42,
""x-test"": ""test""
}
";
var schema = OpenApiModelFactory.Parse<OpenApiSchema>(input, OpenApiSpecVersion.OpenApi3_1, out _, "json");
schema.UnrecognizedKeywords.Should().HaveCount(2);
}

}
}
Original file line numberDiff line numberDiff line change
Expand Up@@ -28,14 +28,3 @@ required:
- name

$dynamicAnchor: "addressDef"
definitions:
address:
$dynamicAnchor: "addressDef"
type: "object"
properties:
street:
type: "string"
city:
type: "string"
postalCode:
type: "string"
41 changes: 34 additions & 7 deletions test/Microsoft.OpenApi.Tests/Models/OpenApiSchemaTests.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -22,7 +22,7 @@ namespace Microsoft.OpenApi.Tests.Models
[Collection("DefaultSettings")]
public class OpenApiSchemaTests
{
public static OpenApiSchema BasicSchema = new();
private static readonly OpenApiSchema BasicSchema = new();

public static readonly OpenApiSchema AdvancedSchemaNumber = new()
{
Expand DownExpand Up@@ -602,15 +602,42 @@ public void OpenApiWalkerVisitsOpenApiSchemaNot()
// Assert
visitor.Titles.Count.Should().Be(2);
}
}

internal class SchemaVisitor : OpenApiVisitorBase
{
public List<string> Titles = new();
[Fact]
public void SerializeSchemaWithUnrecognizedPropertiesWorks()
{
// Arrange
var schema = new OpenApiSchema
{
UnrecognizedKeywords = new Dictionary<string, JsonNode>()
{
["customKeyWord"] = "bar",
["anotherKeyword"] = 42
}
};

public override void Visit(OpenApiSchema schema)
var expected = @"{
""unrecognizedKeywords"": {
""customKeyWord"": ""bar"",
""anotherKeyword"": 42
}
}";

// Act
var actual = schema.SerializeAsJson(OpenApiSpecVersion.OpenApi3_1);

// Assert
actual.MakeLineBreaksEnvironmentNeutral().Should().Be(expected.MakeLineBreaksEnvironmentNeutral());
}

internal class SchemaVisitor : OpenApiVisitorBase
{
Titles.Add(schema.Title);
public List<string> Titles = new();

public override void Visit(OpenApiSchema schema)
{
Titles.Add(schema.Title);
}
}
}
}
Original file line numberDiff line numberDiff line change
Expand Up@@ -512,6 +512,7 @@ namespace Microsoft.OpenApi.Models
public const string Type = "type";
public const string UnevaluatedProperties = "unevaluatedProperties";
public const string UniqueItems = "uniqueItems";
public const string UnrecognizedKeywords = "unrecognizedKeywords";
public const string Url = "url";
public const string V2ReferenceUri = "https://registry/definitions/";
public const string V31ExclusiveMaximum = "exclusiveMaximum";
Expand DownExpand Up@@ -925,6 +926,7 @@ namespace Microsoft.OpenApi.Models
public virtual bool UnEvaluatedProperties { get; set; }
public virtual bool UnevaluatedProperties { get; set; }
public virtual bool? UniqueItems { get; set; }
public virtual System.Collections.Generic.IDictionary<string, System.Text.Json.Nodes.JsonNode> UnrecognizedKeywords { get; set; }
public virtual bool UnresolvedReference { get; set; }
public virtual decimal? V31ExclusiveMaximum { get; set; }
public virtual decimal? V31ExclusiveMinimum { get; set; }
Expand DownExpand Up@@ -1859,6 +1861,7 @@ namespace Microsoft.OpenApi.Writers
public static void WriteOptionalCollection<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IEnumerable<T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, T> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, bool> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, bool> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, string> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, string> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, System.Text.Json.Nodes.JsonNode> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, System.Text.Json.Nodes.JsonNode> action) { }
public static void WriteOptionalMap<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, T> action)
where T : Microsoft.OpenApi.Interfaces.IOpenApiElement { }
public static void WriteOptionalMap<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, string, T> action)
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions src/Microsoft.OpenApi/Models/OpenApiConstants.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -480,6 +480,11 @@ public static class OpenApiConstants
/// </summary>
public const string Properties = "properties";

/// <summary>
/// Field: UnrecognizedKeywords
/// </summary>
public const string UnrecognizedKeywords = "unrecognizedKeywords";

/// <summary>
/// Field: Pattern Properties
/// </summary>
Expand Down
73 changes: 28 additions & 45 deletions src/Microsoft.OpenApi/Models/OpenApiSchema.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -15,12 +15,8 @@ namespace Microsoft.OpenApi.Models
/// <summary>
/// The Schema Object allows the definition of input and output data types.
/// </summary>
public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiReferenceable, IOpenApiSerializable
public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiReferenceable
{
private JsonNode _example;
private JsonNode _default;
private IList<JsonNode> _examples;

/// <summary>
/// Follow JSON Schema definition. Short text providing information about the data.
/// </summary>
Expand DownExpand Up@@ -148,11 +144,7 @@ public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiRe
/// Unlike JSON Schema, the value MUST conform to the defined type for the Schema Object defined at the same level.
/// For example, if type is string, then default can be "foo" but cannot be 1.
/// </summary>
public virtual JsonNode Default
{
get => _default;
set => _default = value;
}
public virtual JsonNode Default { get; set; }

/// <summary>
/// Relevant only for Schema "properties" definitions. Declares the property as "read only".
Expand DownExpand Up@@ -273,22 +265,14 @@ public virtual JsonNode Default
/// To represent examples that cannot be naturally represented in JSON or YAML,
/// a string value can be used to contain the example with escaping where necessary.
/// </summary>
public virtual JsonNode Example
{
get => _example;
set => _example = value;
}
public virtual JsonNode Example { get; set; }

/// <summary>
/// A free-form property to include examples of an instance for this schema.
/// To represent examples that cannot be naturally represented in JSON or YAML,
/// a list of values can be used to contain the examples with escaping where necessary.
/// </summary>
public virtual IList<JsonNode> Examples
{
get => _examples;
set => _examples = value;
}
public virtual IList<JsonNode> Examples { get; set; }

/// <summary>
/// Follow JSON Schema definition: https://tools.ietf.org/html/draft-fge-json-schema-validation-00
Expand DownExpand Up@@ -327,6 +311,11 @@ public virtual IList<JsonNode> Examples
/// </summary>
public virtual IDictionary<string, IOpenApiExtension> Extensions { get; set; } = new Dictionary<string, IOpenApiExtension>();

/// <summary>
/// This object stores any unrecognized keywords found in the schema.
/// </summary>
public virtual IDictionary<string, JsonNode> UnrecognizedKeywords { get; set; } = new Dictionary<string, JsonNode>();

/// <summary>
/// Indicates object is a placeholder reference to an actual object and does not contain valid data.
/// </summary>
Expand DownExpand Up@@ -373,7 +362,7 @@ public OpenApiSchema(OpenApiSchema schema)
MinLength = schema?.MinLength ?? MinLength;
Pattern = schema?.Pattern ?? Pattern;
MultipleOf = schema?.MultipleOf ?? MultipleOf;
_default = schema?.Default != null ? JsonNodeCloneHelper.Clone(schema?.Default) : null;
Default = schema?.Default != null ? JsonNodeCloneHelper.Clone(schema?.Default) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
ReadOnly = schema?.ReadOnly ?? ReadOnly;
WriteOnly = schema?.WriteOnly ?? WriteOnly;
AllOf = schema?.AllOf != null ? new List<OpenApiSchema>(schema.AllOf) : null;
Expand All@@ -392,8 +381,8 @@ public OpenApiSchema(OpenApiSchema schema)
AdditionalPropertiesAllowed = schema?.AdditionalPropertiesAllowed ?? AdditionalPropertiesAllowed;
AdditionalProperties = schema?.AdditionalProperties != null ? new(schema?.AdditionalProperties) : null;
Discriminator = schema?.Discriminator != null ? new(schema?.Discriminator) : null;
_example = schema?.Example != null ? JsonNodeCloneHelper.Clone(schema?.Example) : null;
_examples = schema?.Examples != null ? new List<JsonNode>(schema.Examples) : null;
Example = schema?.Example != null ? JsonNodeCloneHelper.Clone(schema?.Example) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
Examples = schema?.Examples != null ? new List<JsonNode>(schema.Examples) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
Enum = schema?.Enum != null ? new List<JsonNode>(schema.Enum) : null;
Nullable = schema?.Nullable ?? Nullable;
ExternalDocs = schema?.ExternalDocs != null ? new(schema?.ExternalDocs) : null;
Expand All@@ -403,6 +392,7 @@ public OpenApiSchema(OpenApiSchema schema)
UnresolvedReference = schema?.UnresolvedReference ?? UnresolvedReference;
Reference = schema?.Reference != null ? new(schema?.Reference) : null;
Annotations = schema?.Annotations != null ? new Dictionary<string, object>(schema?.Annotations) : null;
UnrecognizedKeywords = schema?.UnrecognizedKeywords != null ? new Dictionary<string, JsonNode>(schema?.UnrecognizedKeywords) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
}

/// <summary>
Expand DownExpand Up@@ -430,7 +420,7 @@ public void SerializeInternal(IOpenApiWriter writer, OpenApiSpecVersion version,

if (version == OpenApiSpecVersion.OpenApi3_1)
{
WriteV31Properties(writer);
WriteJsonSchemaKeywords(writer);
}

// title
Expand DownExpand Up@@ -554,6 +544,12 @@ public void SerializeInternal(IOpenApiWriter writer, OpenApiSpecVersion version,
// extensions
writer.WriteExtensions(Extensions, version);

// Unrecognized keywords
if (UnrecognizedKeywords.Any())
{
writer.WriteOptionalMap(OpenApiConstants.UnrecognizedKeywords, UnrecognizedKeywords, (w,s) => w.WriteAny(s));
}

writer.WriteEndObject();
}

Expand All@@ -564,7 +560,7 @@ public virtual void SerializeAsV2(IOpenApiWriter writer)
SerializeAsV2(writer: writer, parentRequiredProperties: new HashSet<string>(), propertyName: null);
}

internal void WriteV31Properties(IOpenApiWriter writer)
internal void WriteJsonSchemaKeywords(IOpenApiWriter writer)
{
writer.WriteProperty(OpenApiConstants.Id, Id);
writer.WriteProperty(OpenApiConstants.DollarSchema, Schema);
Expand All@@ -577,7 +573,7 @@ internal void WriteV31Properties(IOpenApiWriter writer)
writer.WriteProperty(OpenApiConstants.V31ExclusiveMaximum, V31ExclusiveMaximum);
writer.WriteProperty(OpenApiConstants.V31ExclusiveMinimum, V31ExclusiveMinimum);
writer.WriteProperty(OpenApiConstants.UnevaluatedProperties, UnevaluatedProperties, false);
writer.WriteOptionalCollection(OpenApiConstants.Examples, _examples, (nodeWriter, s) => nodeWriter.WriteAny(s));
writer.WriteOptionalCollection(OpenApiConstants.Examples, Examples, (nodeWriter, s) => nodeWriter.WriteAny(s));
writer.WriteOptionalMap(OpenApiConstants.PatternProperties, PatternProperties, (w, s) => s.SerializeAsV31(w));
}

Expand DownExpand Up@@ -820,15 +816,9 @@ private void SerializeTypeProperty(JsonSchemaType? type, IOpenApiWriter writer,
}
else
{
var list = new List<JsonSchemaType>();
foreach (JsonSchemaType flag in jsonSchemaTypeValues)
{
if (type.Value.HasFlag(flag))
{
list.Add(flag);
}
}

var list = (from JsonSchemaType flag in jsonSchemaTypeValues
where type.Value.HasFlag(flag)
select flag).ToList();
writer.WriteOptionalCollection(OpenApiConstants.Type, list, (w, s) => w.WriteValue(s.ToIdentifier()));
}
}
Expand All@@ -850,16 +840,9 @@ private void UpCastSchemaTypeToV31(JsonSchemaType? type, IOpenApiWriter writer)
{
// create a new array and insert the type and "null" as values
Type = type | JsonSchemaType.Null;
var list = new List<string>();
foreach (JsonSchemaType? flag in jsonSchemaTypeValues)
{
// Check if the flag is set in 'type' using a bitwise AND operation
if (Type.Value.HasFlag(flag))
{
list.Add(flag.ToIdentifier());
}
}

var list = (from JsonSchemaType? flag in jsonSchemaTypeValues// Check if the flag is set in 'type' using a bitwise AND operation
where Type.Value.HasFlag(flag)
select flag.ToIdentifier()).ToList();
writer.WriteOptionalCollection(OpenApiConstants.Type, list, (w, s) => w.WriteValue(s));
}

Expand Down
14 changes: 13 additions & 1 deletion src/Microsoft.OpenApi/Reader/V31/OpenApiSchemaDeserializer.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -7,6 +7,8 @@
using Microsoft.OpenApi.Reader.ParseNodes;
using System.Collections.Generic;
using System.Globalization;
using System.Linq;
using System.Text.Json.Nodes;

namespace Microsoft.OpenApi.Reader.V31
{
Expand DownExpand Up@@ -254,7 +256,17 @@ public static OpenApiSchema LoadSchema(ParseNode node, OpenApiDocument hostDocum

foreach (var propertyNode in mapNode)
{
propertyNode.ParseField(schema, _openApiSchemaFixedFields, _openApiSchemaPatternFields);
bool isRecognized = _openApiSchemaFixedFields.ContainsKey(propertyNode.Name) ||
_openApiSchemaPatternFields.Any(p => p.Key(propertyNode.Name));
Comment thread
baywet marked this conversation as resolved.

if (isRecognized)
{
propertyNode.ParseField(schema, _openApiSchemaFixedFields, _openApiSchemaPatternFields);
}
else
{
schema.UnrecognizedKeywords[propertyNode.Name] = propertyNode.JsonNode;
}
}

if (schema.Extensions.ContainsKey(OpenApiConstants.NullableExtension))
Expand Down
20 changes: 20 additions & 0 deletions src/Microsoft.OpenApi/Writers/OpenApiWriterExtensions.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -5,6 +5,7 @@
using System.Collections;
using System.Collections.Generic;
using System.Linq;
using System.Text.Json.Nodes;
using Microsoft.OpenApi.Interfaces;

namespace Microsoft.OpenApi.Writers
Expand DownExpand Up@@ -253,6 +254,25 @@ public static void WriteRequiredMap(
writer.WriteMapInternal(name, elements, action);
}

/// <summary>
/// Write the optional Open API element map (string to string mapping).
/// </summary>
/// <param name="writer">The Open API writer.</param>
/// <param name="name">The property name.</param>
/// <param name="elements">The map values.</param>
/// <param name="action">The map element writer action.</param>
public static void WriteOptionalMap(
this IOpenApiWriter writer,
string name,
IDictionary<string, JsonNode> elements,
Action<IOpenApiWriter, JsonNode> action)
{
if (elements != null && elements.Any())
{
writer.WriteMapInternal(name, elements, action);
}
}

/// <summary>
/// Write the optional Open API element map (string to string mapping).
/// </summary>
Expand Down
Original file line numberDiff line numberDiff line change
Expand Up@@ -495,5 +495,21 @@ public void ParseSchemaWithConstWorks()
var schemaString = writer.ToString();
schemaString.MakeLineBreaksEnvironmentNeutral().Should().Be(expected.MakeLineBreaksEnvironmentNeutral());
}

[Fact]
public void ParseSchemaWithUnrecognizedKeywordsWorks()
{
var input = @"{
""type"": ""string"",
""format"": ""date-time"",
""customKeyword"": ""customValue"",
""anotherKeyword"": 42,
""x-test"": ""test""
}
";
var schema = OpenApiModelFactory.Parse<OpenApiSchema>(input, OpenApiSpecVersion.OpenApi3_1, out _, "json");
schema.UnrecognizedKeywords.Should().HaveCount(2);
}

}
}
Original file line numberDiff line numberDiff line change
Expand Up@@ -28,14 +28,3 @@ required:
- name

$dynamicAnchor: "addressDef"
definitions:
address:
$dynamicAnchor: "addressDef"
type: "object"
properties:
street:
type: "string"
city:
type: "string"
postalCode:
type: "string"
41 changes: 34 additions & 7 deletions test/Microsoft.OpenApi.Tests/Models/OpenApiSchemaTests.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -22,7 +22,7 @@ namespace Microsoft.OpenApi.Tests.Models
[Collection("DefaultSettings")]
public class OpenApiSchemaTests
{
public static OpenApiSchema BasicSchema = new();
private static readonly OpenApiSchema BasicSchema = new();

public static readonly OpenApiSchema AdvancedSchemaNumber = new()
{
Expand DownExpand Up@@ -602,15 +602,42 @@ public void OpenApiWalkerVisitsOpenApiSchemaNot()
// Assert
visitor.Titles.Count.Should().Be(2);
}
}

internal class SchemaVisitor : OpenApiVisitorBase
{
public List<string> Titles = new();
[Fact]
public void SerializeSchemaWithUnrecognizedPropertiesWorks()
{
// Arrange
var schema = new OpenApiSchema
{
UnrecognizedKeywords = new Dictionary<string, JsonNode>()
{
["customKeyWord"] = "bar",
["anotherKeyword"] = 42
}
};

public override void Visit(OpenApiSchema schema)
var expected = @"{
""unrecognizedKeywords"": {
""customKeyWord"": ""bar"",
""anotherKeyword"": 42
}
}";

// Act
var actual = schema.SerializeAsJson(OpenApiSpecVersion.OpenApi3_1);

// Assert
actual.MakeLineBreaksEnvironmentNeutral().Should().Be(expected.MakeLineBreaksEnvironmentNeutral());
}

internal class SchemaVisitor : OpenApiVisitorBase
{
Titles.Add(schema.Title);
public List<string> Titles = new();

public override void Visit(OpenApiSchema schema)
{
Titles.Add(schema.Title);
}
}
}
}
Original file line numberDiff line numberDiff line change
Expand Up@@ -512,6 +512,7 @@ namespace Microsoft.OpenApi.Models
public const string Type = "type";
public const string UnevaluatedProperties = "unevaluatedProperties";
public const string UniqueItems = "uniqueItems";
public const string UnrecognizedKeywords = "unrecognizedKeywords";
public const string Url = "url";
public const string V2ReferenceUri = "https://registry/definitions/";
public const string V31ExclusiveMaximum = "exclusiveMaximum";
Expand DownExpand Up@@ -925,6 +926,7 @@ namespace Microsoft.OpenApi.Models
public virtual bool UnEvaluatedProperties { get; set; }
public virtual bool UnevaluatedProperties { get; set; }
public virtual bool? UniqueItems { get; set; }
public virtual System.Collections.Generic.IDictionary<string, System.Text.Json.Nodes.JsonNode> UnrecognizedKeywords { get; set; }
public virtual bool UnresolvedReference { get; set; }
public virtual decimal? V31ExclusiveMaximum { get; set; }
public virtual decimal? V31ExclusiveMinimum { get; set; }
Expand DownExpand Up@@ -1859,6 +1861,7 @@ namespace Microsoft.OpenApi.Writers
public static void WriteOptionalCollection<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IEnumerable<T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, T> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, bool> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, bool> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, string> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, string> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, System.Text.Json.Nodes.JsonNode> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, System.Text.Json.Nodes.JsonNode> action) { }
public static void WriteOptionalMap<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, T> action)
where T : Microsoft.OpenApi.Interfaces.IOpenApiElement { }
public static void WriteOptionalMap<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, string, T> action)
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions src/Microsoft.OpenApi/Models/OpenApiConstants.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -480,6 +480,11 @@ public static class OpenApiConstants
/// </summary>
public const string Properties = "properties";

/// <summary>
/// Field: UnrecognizedKeywords
/// </summary>
public const string UnrecognizedKeywords = "unrecognizedKeywords";

/// <summary>
/// Field: Pattern Properties
/// </summary>
Expand Down
73 changes: 28 additions & 45 deletions src/Microsoft.OpenApi/Models/OpenApiSchema.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -15,12 +15,8 @@ namespace Microsoft.OpenApi.Models
/// <summary>
/// The Schema Object allows the definition of input and output data types.
/// </summary>
public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiReferenceable, IOpenApiSerializable
public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiReferenceable
{
private JsonNode _example;
private JsonNode _default;
private IList<JsonNode> _examples;

/// <summary>
/// Follow JSON Schema definition. Short text providing information about the data.
/// </summary>
Expand DownExpand Up@@ -148,11 +144,7 @@ public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiRe
/// Unlike JSON Schema, the value MUST conform to the defined type for the Schema Object defined at the same level.
/// For example, if type is string, then default can be "foo" but cannot be 1.
/// </summary>
public virtual JsonNode Default
{
get => _default;
set => _default = value;
}
public virtual JsonNode Default { get; set; }

/// <summary>
/// Relevant only for Schema "properties" definitions. Declares the property as "read only".
Expand DownExpand Up@@ -273,22 +265,14 @@ public virtual JsonNode Default
/// To represent examples that cannot be naturally represented in JSON or YAML,
/// a string value can be used to contain the example with escaping where necessary.
/// </summary>
public virtual JsonNode Example
{
get => _example;
set => _example = value;
}
public virtual JsonNode Example { get; set; }

/// <summary>
/// A free-form property to include examples of an instance for this schema.
/// To represent examples that cannot be naturally represented in JSON or YAML,
/// a list of values can be used to contain the examples with escaping where necessary.
/// </summary>
public virtual IList<JsonNode> Examples
{
get => _examples;
set => _examples = value;
}
public virtual IList<JsonNode> Examples { get; set; }

/// <summary>
/// Follow JSON Schema definition: https://tools.ietf.org/html/draft-fge-json-schema-validation-00
Expand DownExpand Up@@ -327,6 +311,11 @@ public virtual IList<JsonNode> Examples
/// </summary>
public virtual IDictionary<string, IOpenApiExtension> Extensions { get; set; } = new Dictionary<string, IOpenApiExtension>();

/// <summary>
/// This object stores any unrecognized keywords found in the schema.
/// </summary>
public virtual IDictionary<string, JsonNode> UnrecognizedKeywords { get; set; } = new Dictionary<string, JsonNode>();

/// <summary>
/// Indicates object is a placeholder reference to an actual object and does not contain valid data.
/// </summary>
Expand DownExpand Up@@ -373,7 +362,7 @@ public OpenApiSchema(OpenApiSchema schema)
MinLength = schema?.MinLength ?? MinLength;
Pattern = schema?.Pattern ?? Pattern;
MultipleOf = schema?.MultipleOf ?? MultipleOf;
_default = schema?.Default != null ? JsonNodeCloneHelper.Clone(schema?.Default) : null;
Default = schema?.Default != null ? JsonNodeCloneHelper.Clone(schema?.Default) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
ReadOnly = schema?.ReadOnly ?? ReadOnly;
WriteOnly = schema?.WriteOnly ?? WriteOnly;
AllOf = schema?.AllOf != null ? new List<OpenApiSchema>(schema.AllOf) : null;
Expand All@@ -392,8 +381,8 @@ public OpenApiSchema(OpenApiSchema schema)
AdditionalPropertiesAllowed = schema?.AdditionalPropertiesAllowed ?? AdditionalPropertiesAllowed;
AdditionalProperties = schema?.AdditionalProperties != null ? new(schema?.AdditionalProperties) : null;
Discriminator = schema?.Discriminator != null ? new(schema?.Discriminator) : null;
_example = schema?.Example != null ? JsonNodeCloneHelper.Clone(schema?.Example) : null;
_examples = schema?.Examples != null ? new List<JsonNode>(schema.Examples) : null;
Example = schema?.Example != null ? JsonNodeCloneHelper.Clone(schema?.Example) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
Examples = schema?.Examples != null ? new List<JsonNode>(schema.Examples) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
Enum = schema?.Enum != null ? new List<JsonNode>(schema.Enum) : null;
Nullable = schema?.Nullable ?? Nullable;
ExternalDocs = schema?.ExternalDocs != null ? new(schema?.ExternalDocs) : null;
Expand All@@ -403,6 +392,7 @@ public OpenApiSchema(OpenApiSchema schema)
UnresolvedReference = schema?.UnresolvedReference ?? UnresolvedReference;
Reference = schema?.Reference != null ? new(schema?.Reference) : null;
Annotations = schema?.Annotations != null ? new Dictionary<string, object>(schema?.Annotations) : null;
UnrecognizedKeywords = schema?.UnrecognizedKeywords != null ? new Dictionary<string, JsonNode>(schema?.UnrecognizedKeywords) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
}

/// <summary>
Expand DownExpand Up@@ -430,7 +420,7 @@ public void SerializeInternal(IOpenApiWriter writer, OpenApiSpecVersion version,

if (version == OpenApiSpecVersion.OpenApi3_1)
{
WriteV31Properties(writer);
WriteJsonSchemaKeywords(writer);
}

// title
Expand DownExpand Up@@ -554,6 +544,12 @@ public void SerializeInternal(IOpenApiWriter writer, OpenApiSpecVersion version,
// extensions
writer.WriteExtensions(Extensions, version);

// Unrecognized keywords
if (UnrecognizedKeywords.Any())
{
writer.WriteOptionalMap(OpenApiConstants.UnrecognizedKeywords, UnrecognizedKeywords, (w,s) => w.WriteAny(s));
}

writer.WriteEndObject();
}

Expand All@@ -564,7 +560,7 @@ public virtual void SerializeAsV2(IOpenApiWriter writer)
SerializeAsV2(writer: writer, parentRequiredProperties: new HashSet<string>(), propertyName: null);
}

internal void WriteV31Properties(IOpenApiWriter writer)
internal void WriteJsonSchemaKeywords(IOpenApiWriter writer)
{
writer.WriteProperty(OpenApiConstants.Id, Id);
writer.WriteProperty(OpenApiConstants.DollarSchema, Schema);
Expand All@@ -577,7 +573,7 @@ internal void WriteV31Properties(IOpenApiWriter writer)
writer.WriteProperty(OpenApiConstants.V31ExclusiveMaximum, V31ExclusiveMaximum);
writer.WriteProperty(OpenApiConstants.V31ExclusiveMinimum, V31ExclusiveMinimum);
writer.WriteProperty(OpenApiConstants.UnevaluatedProperties, UnevaluatedProperties, false);
writer.WriteOptionalCollection(OpenApiConstants.Examples, _examples, (nodeWriter, s) => nodeWriter.WriteAny(s));
writer.WriteOptionalCollection(OpenApiConstants.Examples, Examples, (nodeWriter, s) => nodeWriter.WriteAny(s));
writer.WriteOptionalMap(OpenApiConstants.PatternProperties, PatternProperties, (w, s) => s.SerializeAsV31(w));
}

Expand DownExpand Up@@ -820,15 +816,9 @@ private void SerializeTypeProperty(JsonSchemaType? type, IOpenApiWriter writer,
}
else
{
var list = new List<JsonSchemaType>();
foreach (JsonSchemaType flag in jsonSchemaTypeValues)
{
if (type.Value.HasFlag(flag))
{
list.Add(flag);
}
}

var list = (from JsonSchemaType flag in jsonSchemaTypeValues
where type.Value.HasFlag(flag)
select flag).ToList();
writer.WriteOptionalCollection(OpenApiConstants.Type, list, (w, s) => w.WriteValue(s.ToIdentifier()));
}
}
Expand All@@ -850,16 +840,9 @@ private void UpCastSchemaTypeToV31(JsonSchemaType? type, IOpenApiWriter writer)
{
// create a new array and insert the type and "null" as values
Type = type | JsonSchemaType.Null;
var list = new List<string>();
foreach (JsonSchemaType? flag in jsonSchemaTypeValues)
{
// Check if the flag is set in 'type' using a bitwise AND operation
if (Type.Value.HasFlag(flag))
{
list.Add(flag.ToIdentifier());
}
}

var list = (from JsonSchemaType? flag in jsonSchemaTypeValues// Check if the flag is set in 'type' using a bitwise AND operation
where Type.Value.HasFlag(flag)
select flag.ToIdentifier()).ToList();
writer.WriteOptionalCollection(OpenApiConstants.Type, list, (w, s) => w.WriteValue(s));
}

Expand Down
14 changes: 13 additions & 1 deletion src/Microsoft.OpenApi/Reader/V31/OpenApiSchemaDeserializer.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -7,6 +7,8 @@
using Microsoft.OpenApi.Reader.ParseNodes;
using System.Collections.Generic;
using System.Globalization;
using System.Linq;
using System.Text.Json.Nodes;

namespace Microsoft.OpenApi.Reader.V31
{
Expand DownExpand Up@@ -254,7 +256,17 @@ public static OpenApiSchema LoadSchema(ParseNode node, OpenApiDocument hostDocum

foreach (var propertyNode in mapNode)
{
propertyNode.ParseField(schema, _openApiSchemaFixedFields, _openApiSchemaPatternFields);
bool isRecognized = _openApiSchemaFixedFields.ContainsKey(propertyNode.Name) ||
_openApiSchemaPatternFields.Any(p => p.Key(propertyNode.Name));
Comment thread
baywet marked this conversation as resolved.

if (isRecognized)
{
propertyNode.ParseField(schema, _openApiSchemaFixedFields, _openApiSchemaPatternFields);
}
else
{
schema.UnrecognizedKeywords[propertyNode.Name] = propertyNode.JsonNode;
}
}

if (schema.Extensions.ContainsKey(OpenApiConstants.NullableExtension))
Expand Down
20 changes: 20 additions & 0 deletions src/Microsoft.OpenApi/Writers/OpenApiWriterExtensions.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -5,6 +5,7 @@
using System.Collections;
using System.Collections.Generic;
using System.Linq;
using System.Text.Json.Nodes;
using Microsoft.OpenApi.Interfaces;

namespace Microsoft.OpenApi.Writers
Expand DownExpand Up@@ -253,6 +254,25 @@ public static void WriteRequiredMap(
writer.WriteMapInternal(name, elements, action);
}

/// <summary>
/// Write the optional Open API element map (string to string mapping).
/// </summary>
/// <param name="writer">The Open API writer.</param>
/// <param name="name">The property name.</param>
/// <param name="elements">The map values.</param>
/// <param name="action">The map element writer action.</param>
public static void WriteOptionalMap(
this IOpenApiWriter writer,
string name,
IDictionary<string, JsonNode> elements,
Action<IOpenApiWriter, JsonNode> action)
{
if (elements != null && elements.Any())
{
writer.WriteMapInternal(name, elements, action);
}
}

/// <summary>
/// Write the optional Open API element map (string to string mapping).
/// </summary>
Expand Down
Original file line numberDiff line numberDiff line change
Expand Up@@ -495,5 +495,21 @@ public void ParseSchemaWithConstWorks()
var schemaString = writer.ToString();
schemaString.MakeLineBreaksEnvironmentNeutral().Should().Be(expected.MakeLineBreaksEnvironmentNeutral());
}

[Fact]
public void ParseSchemaWithUnrecognizedKeywordsWorks()
{
var input = @"{
""type"": ""string"",
""format"": ""date-time"",
""customKeyword"": ""customValue"",
""anotherKeyword"": 42,
""x-test"": ""test""
}
";
var schema = OpenApiModelFactory.Parse<OpenApiSchema>(input, OpenApiSpecVersion.OpenApi3_1, out _, "json");
schema.UnrecognizedKeywords.Should().HaveCount(2);
}

}
}
Original file line numberDiff line numberDiff line change
Expand Up@@ -28,14 +28,3 @@ required:
- name

$dynamicAnchor: "addressDef"
definitions:
address:
$dynamicAnchor: "addressDef"
type: "object"
properties:
street:
type: "string"
city:
type: "string"
postalCode:
type: "string"
41 changes: 34 additions & 7 deletions test/Microsoft.OpenApi.Tests/Models/OpenApiSchemaTests.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -22,7 +22,7 @@ namespace Microsoft.OpenApi.Tests.Models
[Collection("DefaultSettings")]
public class OpenApiSchemaTests
{
public static OpenApiSchema BasicSchema = new();
private static readonly OpenApiSchema BasicSchema = new();

public static readonly OpenApiSchema AdvancedSchemaNumber = new()
{
Expand DownExpand Up@@ -602,15 +602,42 @@ public void OpenApiWalkerVisitsOpenApiSchemaNot()
// Assert
visitor.Titles.Count.Should().Be(2);
}
}

internal class SchemaVisitor : OpenApiVisitorBase
{
public List<string> Titles = new();
[Fact]
public void SerializeSchemaWithUnrecognizedPropertiesWorks()
{
// Arrange
var schema = new OpenApiSchema
{
UnrecognizedKeywords = new Dictionary<string, JsonNode>()
{
["customKeyWord"] = "bar",
["anotherKeyword"] = 42
}
};

public override void Visit(OpenApiSchema schema)
var expected = @"{
""unrecognizedKeywords"": {
""customKeyWord"": ""bar"",
""anotherKeyword"": 42
}
}";

// Act
var actual = schema.SerializeAsJson(OpenApiSpecVersion.OpenApi3_1);

// Assert
actual.MakeLineBreaksEnvironmentNeutral().Should().Be(expected.MakeLineBreaksEnvironmentNeutral());
}

internal class SchemaVisitor : OpenApiVisitorBase
{
Titles.Add(schema.Title);
public List<string> Titles = new();

public override void Visit(OpenApiSchema schema)
{
Titles.Add(schema.Title);
}
}
}
}
Original file line numberDiff line numberDiff line change
Expand Up@@ -512,6 +512,7 @@ namespace Microsoft.OpenApi.Models
public const string Type = "type";
public const string UnevaluatedProperties = "unevaluatedProperties";
public const string UniqueItems = "uniqueItems";
public const string UnrecognizedKeywords = "unrecognizedKeywords";
public const string Url = "url";
public const string V2ReferenceUri = "https://registry/definitions/";
public const string V31ExclusiveMaximum = "exclusiveMaximum";
Expand DownExpand Up@@ -925,6 +926,7 @@ namespace Microsoft.OpenApi.Models
public virtual bool UnEvaluatedProperties { get; set; }
public virtual bool UnevaluatedProperties { get; set; }
public virtual bool? UniqueItems { get; set; }
public virtual System.Collections.Generic.IDictionary<string, System.Text.Json.Nodes.JsonNode> UnrecognizedKeywords { get; set; }
public virtual bool UnresolvedReference { get; set; }
public virtual decimal? V31ExclusiveMaximum { get; set; }
public virtual decimal? V31ExclusiveMinimum { get; set; }
Expand DownExpand Up@@ -1859,6 +1861,7 @@ namespace Microsoft.OpenApi.Writers
public static void WriteOptionalCollection<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IEnumerable<T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, T> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, bool> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, bool> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, string> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, string> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, System.Text.Json.Nodes.JsonNode> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, System.Text.Json.Nodes.JsonNode> action) { }
public static void WriteOptionalMap<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, T> action)
where T : Microsoft.OpenApi.Interfaces.IOpenApiElement { }
public static void WriteOptionalMap<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, string, T> action)
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions src/Microsoft.OpenApi/Models/OpenApiConstants.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -480,6 +480,11 @@ public static class OpenApiConstants
/// </summary>
public const string Properties = "properties";

/// <summary>
/// Field: UnrecognizedKeywords
/// </summary>
public const string UnrecognizedKeywords = "unrecognizedKeywords";

/// <summary>
/// Field: Pattern Properties
/// </summary>
Expand Down
73 changes: 28 additions & 45 deletions src/Microsoft.OpenApi/Models/OpenApiSchema.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -15,12 +15,8 @@ namespace Microsoft.OpenApi.Models
/// <summary>
/// The Schema Object allows the definition of input and output data types.
/// </summary>
public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiReferenceable, IOpenApiSerializable
public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiReferenceable
{
private JsonNode _example;
private JsonNode _default;
private IList<JsonNode> _examples;

/// <summary>
/// Follow JSON Schema definition. Short text providing information about the data.
/// </summary>
Expand DownExpand Up@@ -148,11 +144,7 @@ public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiRe
/// Unlike JSON Schema, the value MUST conform to the defined type for the Schema Object defined at the same level.
/// For example, if type is string, then default can be "foo" but cannot be 1.
/// </summary>
public virtual JsonNode Default
{
get => _default;
set => _default = value;
}
public virtual JsonNode Default { get; set; }

/// <summary>
/// Relevant only for Schema "properties" definitions. Declares the property as "read only".
Expand DownExpand Up@@ -273,22 +265,14 @@ public virtual JsonNode Default
/// To represent examples that cannot be naturally represented in JSON or YAML,
/// a string value can be used to contain the example with escaping where necessary.
/// </summary>
public virtual JsonNode Example
{
get => _example;
set => _example = value;
}
public virtual JsonNode Example { get; set; }

/// <summary>
/// A free-form property to include examples of an instance for this schema.
/// To represent examples that cannot be naturally represented in JSON or YAML,
/// a list of values can be used to contain the examples with escaping where necessary.
/// </summary>
public virtual IList<JsonNode> Examples
{
get => _examples;
set => _examples = value;
}
public virtual IList<JsonNode> Examples { get; set; }

/// <summary>
/// Follow JSON Schema definition: https://tools.ietf.org/html/draft-fge-json-schema-validation-00
Expand DownExpand Up@@ -327,6 +311,11 @@ public virtual IList<JsonNode> Examples
/// </summary>
public virtual IDictionary<string, IOpenApiExtension> Extensions { get; set; } = new Dictionary<string, IOpenApiExtension>();

/// <summary>
/// This object stores any unrecognized keywords found in the schema.
/// </summary>
public virtual IDictionary<string, JsonNode> UnrecognizedKeywords { get; set; } = new Dictionary<string, JsonNode>();

/// <summary>
/// Indicates object is a placeholder reference to an actual object and does not contain valid data.
/// </summary>
Expand DownExpand Up@@ -373,7 +362,7 @@ public OpenApiSchema(OpenApiSchema schema)
MinLength = schema?.MinLength ?? MinLength;
Pattern = schema?.Pattern ?? Pattern;
MultipleOf = schema?.MultipleOf ?? MultipleOf;
_default = schema?.Default != null ? JsonNodeCloneHelper.Clone(schema?.Default) : null;
Default = schema?.Default != null ? JsonNodeCloneHelper.Clone(schema?.Default) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
ReadOnly = schema?.ReadOnly ?? ReadOnly;
WriteOnly = schema?.WriteOnly ?? WriteOnly;
AllOf = schema?.AllOf != null ? new List<OpenApiSchema>(schema.AllOf) : null;
Expand All@@ -392,8 +381,8 @@ public OpenApiSchema(OpenApiSchema schema)
AdditionalPropertiesAllowed = schema?.AdditionalPropertiesAllowed ?? AdditionalPropertiesAllowed;
AdditionalProperties = schema?.AdditionalProperties != null ? new(schema?.AdditionalProperties) : null;
Discriminator = schema?.Discriminator != null ? new(schema?.Discriminator) : null;
_example = schema?.Example != null ? JsonNodeCloneHelper.Clone(schema?.Example) : null;
_examples = schema?.Examples != null ? new List<JsonNode>(schema.Examples) : null;
Example = schema?.Example != null ? JsonNodeCloneHelper.Clone(schema?.Example) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
Examples = schema?.Examples != null ? new List<JsonNode>(schema.Examples) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
Enum = schema?.Enum != null ? new List<JsonNode>(schema.Enum) : null;
Nullable = schema?.Nullable ?? Nullable;
ExternalDocs = schema?.ExternalDocs != null ? new(schema?.ExternalDocs) : null;
Expand All@@ -403,6 +392,7 @@ public OpenApiSchema(OpenApiSchema schema)
UnresolvedReference = schema?.UnresolvedReference ?? UnresolvedReference;
Reference = schema?.Reference != null ? new(schema?.Reference) : null;
Annotations = schema?.Annotations != null ? new Dictionary<string, object>(schema?.Annotations) : null;
UnrecognizedKeywords = schema?.UnrecognizedKeywords != null ? new Dictionary<string, JsonNode>(schema?.UnrecognizedKeywords) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
}

/// <summary>
Expand DownExpand Up@@ -430,7 +420,7 @@ public void SerializeInternal(IOpenApiWriter writer, OpenApiSpecVersion version,

if (version == OpenApiSpecVersion.OpenApi3_1)
{
WriteV31Properties(writer);
WriteJsonSchemaKeywords(writer);
}

// title
Expand DownExpand Up@@ -554,6 +544,12 @@ public void SerializeInternal(IOpenApiWriter writer, OpenApiSpecVersion version,
// extensions
writer.WriteExtensions(Extensions, version);

// Unrecognized keywords
if (UnrecognizedKeywords.Any())
{
writer.WriteOptionalMap(OpenApiConstants.UnrecognizedKeywords, UnrecognizedKeywords, (w,s) => w.WriteAny(s));
}

writer.WriteEndObject();
}

Expand All@@ -564,7 +560,7 @@ public virtual void SerializeAsV2(IOpenApiWriter writer)
SerializeAsV2(writer: writer, parentRequiredProperties: new HashSet<string>(), propertyName: null);
}

internal void WriteV31Properties(IOpenApiWriter writer)
internal void WriteJsonSchemaKeywords(IOpenApiWriter writer)
{
writer.WriteProperty(OpenApiConstants.Id, Id);
writer.WriteProperty(OpenApiConstants.DollarSchema, Schema);
Expand All@@ -577,7 +573,7 @@ internal void WriteV31Properties(IOpenApiWriter writer)
writer.WriteProperty(OpenApiConstants.V31ExclusiveMaximum, V31ExclusiveMaximum);
writer.WriteProperty(OpenApiConstants.V31ExclusiveMinimum, V31ExclusiveMinimum);
writer.WriteProperty(OpenApiConstants.UnevaluatedProperties, UnevaluatedProperties, false);
writer.WriteOptionalCollection(OpenApiConstants.Examples, _examples, (nodeWriter, s) => nodeWriter.WriteAny(s));
writer.WriteOptionalCollection(OpenApiConstants.Examples, Examples, (nodeWriter, s) => nodeWriter.WriteAny(s));
writer.WriteOptionalMap(OpenApiConstants.PatternProperties, PatternProperties, (w, s) => s.SerializeAsV31(w));
}

Expand DownExpand Up@@ -820,15 +816,9 @@ private void SerializeTypeProperty(JsonSchemaType? type, IOpenApiWriter writer,
}
else
{
var list = new List<JsonSchemaType>();
foreach (JsonSchemaType flag in jsonSchemaTypeValues)
{
if (type.Value.HasFlag(flag))
{
list.Add(flag);
}
}

var list = (from JsonSchemaType flag in jsonSchemaTypeValues
where type.Value.HasFlag(flag)
select flag).ToList();
writer.WriteOptionalCollection(OpenApiConstants.Type, list, (w, s) => w.WriteValue(s.ToIdentifier()));
}
}
Expand All@@ -850,16 +840,9 @@ private void UpCastSchemaTypeToV31(JsonSchemaType? type, IOpenApiWriter writer)
{
// create a new array and insert the type and "null" as values
Type = type | JsonSchemaType.Null;
var list = new List<string>();
foreach (JsonSchemaType? flag in jsonSchemaTypeValues)
{
// Check if the flag is set in 'type' using a bitwise AND operation
if (Type.Value.HasFlag(flag))
{
list.Add(flag.ToIdentifier());
}
}

var list = (from JsonSchemaType? flag in jsonSchemaTypeValues// Check if the flag is set in 'type' using a bitwise AND operation
where Type.Value.HasFlag(flag)
select flag.ToIdentifier()).ToList();
writer.WriteOptionalCollection(OpenApiConstants.Type, list, (w, s) => w.WriteValue(s));
}

Expand Down
14 changes: 13 additions & 1 deletion src/Microsoft.OpenApi/Reader/V31/OpenApiSchemaDeserializer.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -7,6 +7,8 @@
using Microsoft.OpenApi.Reader.ParseNodes;
using System.Collections.Generic;
using System.Globalization;
using System.Linq;
using System.Text.Json.Nodes;

namespace Microsoft.OpenApi.Reader.V31
{
Expand DownExpand Up@@ -254,7 +256,17 @@ public static OpenApiSchema LoadSchema(ParseNode node, OpenApiDocument hostDocum

foreach (var propertyNode in mapNode)
{
propertyNode.ParseField(schema, _openApiSchemaFixedFields, _openApiSchemaPatternFields);
bool isRecognized = _openApiSchemaFixedFields.ContainsKey(propertyNode.Name) ||
_openApiSchemaPatternFields.Any(p => p.Key(propertyNode.Name));
Comment thread
baywet marked this conversation as resolved.

if (isRecognized)
{
propertyNode.ParseField(schema, _openApiSchemaFixedFields, _openApiSchemaPatternFields);
}
else
{
schema.UnrecognizedKeywords[propertyNode.Name] = propertyNode.JsonNode;
}
}

if (schema.Extensions.ContainsKey(OpenApiConstants.NullableExtension))
Expand Down
20 changes: 20 additions & 0 deletions src/Microsoft.OpenApi/Writers/OpenApiWriterExtensions.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -5,6 +5,7 @@
using System.Collections;
using System.Collections.Generic;
using System.Linq;
using System.Text.Json.Nodes;
using Microsoft.OpenApi.Interfaces;

namespace Microsoft.OpenApi.Writers
Expand DownExpand Up@@ -253,6 +254,25 @@ public static void WriteRequiredMap(
writer.WriteMapInternal(name, elements, action);
}

/// <summary>
/// Write the optional Open API element map (string to string mapping).
/// </summary>
/// <param name="writer">The Open API writer.</param>
/// <param name="name">The property name.</param>
/// <param name="elements">The map values.</param>
/// <param name="action">The map element writer action.</param>
public static void WriteOptionalMap(
this IOpenApiWriter writer,
string name,
IDictionary<string, JsonNode> elements,
Action<IOpenApiWriter, JsonNode> action)
{
if (elements != null && elements.Any())
{
writer.WriteMapInternal(name, elements, action);
}
}

/// <summary>
/// Write the optional Open API element map (string to string mapping).
/// </summary>
Expand Down
Original file line numberDiff line numberDiff line change
Expand Up@@ -495,5 +495,21 @@ public void ParseSchemaWithConstWorks()
var schemaString = writer.ToString();
schemaString.MakeLineBreaksEnvironmentNeutral().Should().Be(expected.MakeLineBreaksEnvironmentNeutral());
}

[Fact]
public void ParseSchemaWithUnrecognizedKeywordsWorks()
{
var input = @"{
""type"": ""string"",
""format"": ""date-time"",
""customKeyword"": ""customValue"",
""anotherKeyword"": 42,
""x-test"": ""test""
}
";
var schema = OpenApiModelFactory.Parse<OpenApiSchema>(input, OpenApiSpecVersion.OpenApi3_1, out _, "json");
schema.UnrecognizedKeywords.Should().HaveCount(2);
}

}
}
Original file line numberDiff line numberDiff line change
Expand Up@@ -28,14 +28,3 @@ required:
- name

$dynamicAnchor: "addressDef"
definitions:
address:
$dynamicAnchor: "addressDef"
type: "object"
properties:
street:
type: "string"
city:
type: "string"
postalCode:
type: "string"
41 changes: 34 additions & 7 deletions test/Microsoft.OpenApi.Tests/Models/OpenApiSchemaTests.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -22,7 +22,7 @@ namespace Microsoft.OpenApi.Tests.Models
[Collection("DefaultSettings")]
public class OpenApiSchemaTests
{
public static OpenApiSchema BasicSchema = new();
private static readonly OpenApiSchema BasicSchema = new();

public static readonly OpenApiSchema AdvancedSchemaNumber = new()
{
Expand DownExpand Up@@ -602,15 +602,42 @@ public void OpenApiWalkerVisitsOpenApiSchemaNot()
// Assert
visitor.Titles.Count.Should().Be(2);
}
}

internal class SchemaVisitor : OpenApiVisitorBase
{
public List<string> Titles = new();
[Fact]
public void SerializeSchemaWithUnrecognizedPropertiesWorks()
{
// Arrange
var schema = new OpenApiSchema
{
UnrecognizedKeywords = new Dictionary<string, JsonNode>()
{
["customKeyWord"] = "bar",
["anotherKeyword"] = 42
}
};

public override void Visit(OpenApiSchema schema)
var expected = @"{
""unrecognizedKeywords"": {
""customKeyWord"": ""bar"",
""anotherKeyword"": 42
}
}";

// Act
var actual = schema.SerializeAsJson(OpenApiSpecVersion.OpenApi3_1);

// Assert
actual.MakeLineBreaksEnvironmentNeutral().Should().Be(expected.MakeLineBreaksEnvironmentNeutral());
}

internal class SchemaVisitor : OpenApiVisitorBase
{
Titles.Add(schema.Title);
public List<string> Titles = new();

public override void Visit(OpenApiSchema schema)
{
Titles.Add(schema.Title);
}
}
}
}
Original file line numberDiff line numberDiff line change
Expand Up@@ -512,6 +512,7 @@ namespace Microsoft.OpenApi.Models
public const string Type = "type";
public const string UnevaluatedProperties = "unevaluatedProperties";
public const string UniqueItems = "uniqueItems";
public const string UnrecognizedKeywords = "unrecognizedKeywords";
public const string Url = "url";
public const string V2ReferenceUri = "https://registry/definitions/";
public const string V31ExclusiveMaximum = "exclusiveMaximum";
Expand DownExpand Up@@ -925,6 +926,7 @@ namespace Microsoft.OpenApi.Models
public virtual bool UnEvaluatedProperties { get; set; }
public virtual bool UnevaluatedProperties { get; set; }
public virtual bool? UniqueItems { get; set; }
public virtual System.Collections.Generic.IDictionary<string, System.Text.Json.Nodes.JsonNode> UnrecognizedKeywords { get; set; }
public virtual bool UnresolvedReference { get; set; }
public virtual decimal? V31ExclusiveMaximum { get; set; }
public virtual decimal? V31ExclusiveMinimum { get; set; }
Expand DownExpand Up@@ -1859,6 +1861,7 @@ namespace Microsoft.OpenApi.Writers
public static void WriteOptionalCollection<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IEnumerable<T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, T> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, bool> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, bool> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, string> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, string> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, System.Text.Json.Nodes.JsonNode> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, System.Text.Json.Nodes.JsonNode> action) { }
public static void WriteOptionalMap<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, T> action)
where T : Microsoft.OpenApi.Interfaces.IOpenApiElement { }
public static void WriteOptionalMap<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, string, T> action)
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions src/Microsoft.OpenApi/Models/OpenApiConstants.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -480,6 +480,11 @@ public static class OpenApiConstants
/// </summary>
public const string Properties = "properties";

/// <summary>
/// Field: UnrecognizedKeywords
/// </summary>
public const string UnrecognizedKeywords = "unrecognizedKeywords";

/// <summary>
/// Field: Pattern Properties
/// </summary>
Expand Down
73 changes: 28 additions & 45 deletions src/Microsoft.OpenApi/Models/OpenApiSchema.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -15,12 +15,8 @@ namespace Microsoft.OpenApi.Models
/// <summary>
/// The Schema Object allows the definition of input and output data types.
/// </summary>
public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiReferenceable, IOpenApiSerializable
public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiReferenceable
{
private JsonNode _example;
private JsonNode _default;
private IList<JsonNode> _examples;

/// <summary>
/// Follow JSON Schema definition. Short text providing information about the data.
/// </summary>
Expand DownExpand Up@@ -148,11 +144,7 @@ public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiRe
/// Unlike JSON Schema, the value MUST conform to the defined type for the Schema Object defined at the same level.
/// For example, if type is string, then default can be "foo" but cannot be 1.
/// </summary>
public virtual JsonNode Default
{
get => _default;
set => _default = value;
}
public virtual JsonNode Default { get; set; }

/// <summary>
/// Relevant only for Schema "properties" definitions. Declares the property as "read only".
Expand DownExpand Up@@ -273,22 +265,14 @@ public virtual JsonNode Default
/// To represent examples that cannot be naturally represented in JSON or YAML,
/// a string value can be used to contain the example with escaping where necessary.
/// </summary>
public virtual JsonNode Example
{
get => _example;
set => _example = value;
}
public virtual JsonNode Example { get; set; }

/// <summary>
/// A free-form property to include examples of an instance for this schema.
/// To represent examples that cannot be naturally represented in JSON or YAML,
/// a list of values can be used to contain the examples with escaping where necessary.
/// </summary>
public virtual IList<JsonNode> Examples
{
get => _examples;
set => _examples = value;
}
public virtual IList<JsonNode> Examples { get; set; }

/// <summary>
/// Follow JSON Schema definition: https://tools.ietf.org/html/draft-fge-json-schema-validation-00
Expand DownExpand Up@@ -327,6 +311,11 @@ public virtual IList<JsonNode> Examples
/// </summary>
public virtual IDictionary<string, IOpenApiExtension> Extensions { get; set; } = new Dictionary<string, IOpenApiExtension>();

/// <summary>
/// This object stores any unrecognized keywords found in the schema.
/// </summary>
public virtual IDictionary<string, JsonNode> UnrecognizedKeywords { get; set; } = new Dictionary<string, JsonNode>();

/// <summary>
/// Indicates object is a placeholder reference to an actual object and does not contain valid data.
/// </summary>
Expand DownExpand Up@@ -373,7 +362,7 @@ public OpenApiSchema(OpenApiSchema schema)
MinLength = schema?.MinLength ?? MinLength;
Pattern = schema?.Pattern ?? Pattern;
MultipleOf = schema?.MultipleOf ?? MultipleOf;
_default = schema?.Default != null ? JsonNodeCloneHelper.Clone(schema?.Default) : null;
Default = schema?.Default != null ? JsonNodeCloneHelper.Clone(schema?.Default) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
ReadOnly = schema?.ReadOnly ?? ReadOnly;
WriteOnly = schema?.WriteOnly ?? WriteOnly;
AllOf = schema?.AllOf != null ? new List<OpenApiSchema>(schema.AllOf) : null;
Expand All@@ -392,8 +381,8 @@ public OpenApiSchema(OpenApiSchema schema)
AdditionalPropertiesAllowed = schema?.AdditionalPropertiesAllowed ?? AdditionalPropertiesAllowed;
AdditionalProperties = schema?.AdditionalProperties != null ? new(schema?.AdditionalProperties) : null;
Discriminator = schema?.Discriminator != null ? new(schema?.Discriminator) : null;
_example = schema?.Example != null ? JsonNodeCloneHelper.Clone(schema?.Example) : null;
_examples = schema?.Examples != null ? new List<JsonNode>(schema.Examples) : null;
Example = schema?.Example != null ? JsonNodeCloneHelper.Clone(schema?.Example) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
Examples = schema?.Examples != null ? new List<JsonNode>(schema.Examples) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
Enum = schema?.Enum != null ? new List<JsonNode>(schema.Enum) : null;
Nullable = schema?.Nullable ?? Nullable;
ExternalDocs = schema?.ExternalDocs != null ? new(schema?.ExternalDocs) : null;
Expand All@@ -403,6 +392,7 @@ public OpenApiSchema(OpenApiSchema schema)
UnresolvedReference = schema?.UnresolvedReference ?? UnresolvedReference;
Reference = schema?.Reference != null ? new(schema?.Reference) : null;
Annotations = schema?.Annotations != null ? new Dictionary<string, object>(schema?.Annotations) : null;
UnrecognizedKeywords = schema?.UnrecognizedKeywords != null ? new Dictionary<string, JsonNode>(schema?.UnrecognizedKeywords) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
}

/// <summary>
Expand DownExpand Up@@ -430,7 +420,7 @@ public void SerializeInternal(IOpenApiWriter writer, OpenApiSpecVersion version,

if (version == OpenApiSpecVersion.OpenApi3_1)
{
WriteV31Properties(writer);
WriteJsonSchemaKeywords(writer);
}

// title
Expand DownExpand Up@@ -554,6 +544,12 @@ public void SerializeInternal(IOpenApiWriter writer, OpenApiSpecVersion version,
// extensions
writer.WriteExtensions(Extensions, version);

// Unrecognized keywords
if (UnrecognizedKeywords.Any())
{
writer.WriteOptionalMap(OpenApiConstants.UnrecognizedKeywords, UnrecognizedKeywords, (w,s) => w.WriteAny(s));
}

writer.WriteEndObject();
}

Expand All@@ -564,7 +560,7 @@ public virtual void SerializeAsV2(IOpenApiWriter writer)
SerializeAsV2(writer: writer, parentRequiredProperties: new HashSet<string>(), propertyName: null);
}

internal void WriteV31Properties(IOpenApiWriter writer)
internal void WriteJsonSchemaKeywords(IOpenApiWriter writer)
{
writer.WriteProperty(OpenApiConstants.Id, Id);
writer.WriteProperty(OpenApiConstants.DollarSchema, Schema);
Expand All@@ -577,7 +573,7 @@ internal void WriteV31Properties(IOpenApiWriter writer)
writer.WriteProperty(OpenApiConstants.V31ExclusiveMaximum, V31ExclusiveMaximum);
writer.WriteProperty(OpenApiConstants.V31ExclusiveMinimum, V31ExclusiveMinimum);
writer.WriteProperty(OpenApiConstants.UnevaluatedProperties, UnevaluatedProperties, false);
writer.WriteOptionalCollection(OpenApiConstants.Examples, _examples, (nodeWriter, s) => nodeWriter.WriteAny(s));
writer.WriteOptionalCollection(OpenApiConstants.Examples, Examples, (nodeWriter, s) => nodeWriter.WriteAny(s));
writer.WriteOptionalMap(OpenApiConstants.PatternProperties, PatternProperties, (w, s) => s.SerializeAsV31(w));
}

Expand DownExpand Up@@ -820,15 +816,9 @@ private void SerializeTypeProperty(JsonSchemaType? type, IOpenApiWriter writer,
}
else
{
var list = new List<JsonSchemaType>();
foreach (JsonSchemaType flag in jsonSchemaTypeValues)
{
if (type.Value.HasFlag(flag))
{
list.Add(flag);
}
}

var list = (from JsonSchemaType flag in jsonSchemaTypeValues
where type.Value.HasFlag(flag)
select flag).ToList();
writer.WriteOptionalCollection(OpenApiConstants.Type, list, (w, s) => w.WriteValue(s.ToIdentifier()));
}
}
Expand All@@ -850,16 +840,9 @@ private void UpCastSchemaTypeToV31(JsonSchemaType? type, IOpenApiWriter writer)
{
// create a new array and insert the type and "null" as values
Type = type | JsonSchemaType.Null;
var list = new List<string>();
foreach (JsonSchemaType? flag in jsonSchemaTypeValues)
{
// Check if the flag is set in 'type' using a bitwise AND operation
if (Type.Value.HasFlag(flag))
{
list.Add(flag.ToIdentifier());
}
}

var list = (from JsonSchemaType? flag in jsonSchemaTypeValues// Check if the flag is set in 'type' using a bitwise AND operation
where Type.Value.HasFlag(flag)
select flag.ToIdentifier()).ToList();
writer.WriteOptionalCollection(OpenApiConstants.Type, list, (w, s) => w.WriteValue(s));
}

Expand Down
14 changes: 13 additions & 1 deletion src/Microsoft.OpenApi/Reader/V31/OpenApiSchemaDeserializer.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -7,6 +7,8 @@
using Microsoft.OpenApi.Reader.ParseNodes;
using System.Collections.Generic;
using System.Globalization;
using System.Linq;
using System.Text.Json.Nodes;

namespace Microsoft.OpenApi.Reader.V31
{
Expand DownExpand Up@@ -254,7 +256,17 @@ public static OpenApiSchema LoadSchema(ParseNode node, OpenApiDocument hostDocum

foreach (var propertyNode in mapNode)
{
propertyNode.ParseField(schema, _openApiSchemaFixedFields, _openApiSchemaPatternFields);
bool isRecognized = _openApiSchemaFixedFields.ContainsKey(propertyNode.Name) ||
_openApiSchemaPatternFields.Any(p => p.Key(propertyNode.Name));
Comment thread
baywet marked this conversation as resolved.

if (isRecognized)
{
propertyNode.ParseField(schema, _openApiSchemaFixedFields, _openApiSchemaPatternFields);
}
else
{
schema.UnrecognizedKeywords[propertyNode.Name] = propertyNode.JsonNode;
}
}

if (schema.Extensions.ContainsKey(OpenApiConstants.NullableExtension))
Expand Down
20 changes: 20 additions & 0 deletions src/Microsoft.OpenApi/Writers/OpenApiWriterExtensions.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -5,6 +5,7 @@
using System.Collections;
using System.Collections.Generic;
using System.Linq;
using System.Text.Json.Nodes;
using Microsoft.OpenApi.Interfaces;

namespace Microsoft.OpenApi.Writers
Expand DownExpand Up@@ -253,6 +254,25 @@ public static void WriteRequiredMap(
writer.WriteMapInternal(name, elements, action);
}

/// <summary>
/// Write the optional Open API element map (string to string mapping).
/// </summary>
/// <param name="writer">The Open API writer.</param>
/// <param name="name">The property name.</param>
/// <param name="elements">The map values.</param>
/// <param name="action">The map element writer action.</param>
public static void WriteOptionalMap(
this IOpenApiWriter writer,
string name,
IDictionary<string, JsonNode> elements,
Action<IOpenApiWriter, JsonNode> action)
{
if (elements != null && elements.Any())
{
writer.WriteMapInternal(name, elements, action);
}
}

/// <summary>
/// Write the optional Open API element map (string to string mapping).
/// </summary>
Expand Down
Original file line numberDiff line numberDiff line change
Expand Up@@ -495,5 +495,21 @@ public void ParseSchemaWithConstWorks()
var schemaString = writer.ToString();
schemaString.MakeLineBreaksEnvironmentNeutral().Should().Be(expected.MakeLineBreaksEnvironmentNeutral());
}

[Fact]
public void ParseSchemaWithUnrecognizedKeywordsWorks()
{
var input = @"{
""type"": ""string"",
""format"": ""date-time"",
""customKeyword"": ""customValue"",
""anotherKeyword"": 42,
""x-test"": ""test""
}
";
var schema = OpenApiModelFactory.Parse<OpenApiSchema>(input, OpenApiSpecVersion.OpenApi3_1, out _, "json");
schema.UnrecognizedKeywords.Should().HaveCount(2);
}

}
}
Original file line numberDiff line numberDiff line change
Expand Up@@ -28,14 +28,3 @@ required:
- name

$dynamicAnchor: "addressDef"
definitions:
address:
$dynamicAnchor: "addressDef"
type: "object"
properties:
street:
type: "string"
city:
type: "string"
postalCode:
type: "string"
41 changes: 34 additions & 7 deletions test/Microsoft.OpenApi.Tests/Models/OpenApiSchemaTests.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -22,7 +22,7 @@ namespace Microsoft.OpenApi.Tests.Models
[Collection("DefaultSettings")]
public class OpenApiSchemaTests
{
public static OpenApiSchema BasicSchema = new();
private static readonly OpenApiSchema BasicSchema = new();

public static readonly OpenApiSchema AdvancedSchemaNumber = new()
{
Expand DownExpand Up@@ -602,15 +602,42 @@ public void OpenApiWalkerVisitsOpenApiSchemaNot()
// Assert
visitor.Titles.Count.Should().Be(2);
}
}

internal class SchemaVisitor : OpenApiVisitorBase
{
public List<string> Titles = new();
[Fact]
public void SerializeSchemaWithUnrecognizedPropertiesWorks()
{
// Arrange
var schema = new OpenApiSchema
{
UnrecognizedKeywords = new Dictionary<string, JsonNode>()
{
["customKeyWord"] = "bar",
["anotherKeyword"] = 42
}
};

public override void Visit(OpenApiSchema schema)
var expected = @"{
""unrecognizedKeywords"": {
""customKeyWord"": ""bar"",
""anotherKeyword"": 42
}
}";

// Act
var actual = schema.SerializeAsJson(OpenApiSpecVersion.OpenApi3_1);

// Assert
actual.MakeLineBreaksEnvironmentNeutral().Should().Be(expected.MakeLineBreaksEnvironmentNeutral());
}

internal class SchemaVisitor : OpenApiVisitorBase
{
Titles.Add(schema.Title);
public List<string> Titles = new();

public override void Visit(OpenApiSchema schema)
{
Titles.Add(schema.Title);
}
}
}
}
Original file line numberDiff line numberDiff line change
Expand Up@@ -512,6 +512,7 @@ namespace Microsoft.OpenApi.Models
public const string Type = "type";
public const string UnevaluatedProperties = "unevaluatedProperties";
public const string UniqueItems = "uniqueItems";
public const string UnrecognizedKeywords = "unrecognizedKeywords";
public const string Url = "url";
public const string V2ReferenceUri = "https://registry/definitions/";
public const string V31ExclusiveMaximum = "exclusiveMaximum";
Expand DownExpand Up@@ -925,6 +926,7 @@ namespace Microsoft.OpenApi.Models
public virtual bool UnEvaluatedProperties { get; set; }
public virtual bool UnevaluatedProperties { get; set; }
public virtual bool? UniqueItems { get; set; }
public virtual System.Collections.Generic.IDictionary<string, System.Text.Json.Nodes.JsonNode> UnrecognizedKeywords { get; set; }
public virtual bool UnresolvedReference { get; set; }
public virtual decimal? V31ExclusiveMaximum { get; set; }
public virtual decimal? V31ExclusiveMinimum { get; set; }
Expand DownExpand Up@@ -1859,6 +1861,7 @@ namespace Microsoft.OpenApi.Writers
public static void WriteOptionalCollection<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IEnumerable<T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, T> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, bool> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, bool> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, string> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, string> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, System.Text.Json.Nodes.JsonNode> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, System.Text.Json.Nodes.JsonNode> action) { }
public static void WriteOptionalMap<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, T> action)
where T : Microsoft.OpenApi.Interfaces.IOpenApiElement { }
public static void WriteOptionalMap<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, string, T> action)
Expand Down
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions src/Microsoft.OpenApi/Models/OpenApiConstants.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -480,6 +480,11 @@ public static class OpenApiConstants
/// </summary>
public const string Properties = "properties";

/// <summary>
/// Field: UnrecognizedKeywords
/// </summary>
public const string UnrecognizedKeywords = "unrecognizedKeywords";

/// <summary>
/// Field: Pattern Properties
/// </summary>
Expand Down
73 changes: 28 additions & 45 deletions src/Microsoft.OpenApi/Models/OpenApiSchema.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -15,12 +15,8 @@ namespace Microsoft.OpenApi.Models
/// <summary>
/// The Schema Object allows the definition of input and output data types.
/// </summary>
public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiReferenceable, IOpenApiSerializable
public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiReferenceable
{
private JsonNode _example;
private JsonNode _default;
private IList<JsonNode> _examples;

/// <summary>
/// Follow JSON Schema definition. Short text providing information about the data.
/// </summary>
Expand DownExpand Up@@ -148,11 +144,7 @@ public class OpenApiSchema : IOpenApiAnnotatable, IOpenApiExtensible, IOpenApiRe
/// Unlike JSON Schema, the value MUST conform to the defined type for the Schema Object defined at the same level.
/// For example, if type is string, then default can be "foo" but cannot be 1.
/// </summary>
public virtual JsonNode Default
{
get => _default;
set => _default = value;
}
public virtual JsonNode Default { get; set; }

/// <summary>
/// Relevant only for Schema "properties" definitions. Declares the property as "read only".
Expand DownExpand Up@@ -273,22 +265,14 @@ public virtual JsonNode Default
/// To represent examples that cannot be naturally represented in JSON or YAML,
/// a string value can be used to contain the example with escaping where necessary.
/// </summary>
public virtual JsonNode Example
{
get => _example;
set => _example = value;
}
public virtual JsonNode Example { get; set; }

/// <summary>
/// A free-form property to include examples of an instance for this schema.
/// To represent examples that cannot be naturally represented in JSON or YAML,
/// a list of values can be used to contain the examples with escaping where necessary.
/// </summary>
public virtual IList<JsonNode> Examples
{
get => _examples;
set => _examples = value;
}
public virtual IList<JsonNode> Examples { get; set; }

/// <summary>
/// Follow JSON Schema definition: https://tools.ietf.org/html/draft-fge-json-schema-validation-00
Expand DownExpand Up@@ -327,6 +311,11 @@ public virtual IList<JsonNode> Examples
/// </summary>
public virtual IDictionary<string, IOpenApiExtension> Extensions { get; set; } = new Dictionary<string, IOpenApiExtension>();

/// <summary>
/// This object stores any unrecognized keywords found in the schema.
/// </summary>
public virtual IDictionary<string, JsonNode> UnrecognizedKeywords { get; set; } = new Dictionary<string, JsonNode>();

/// <summary>
/// Indicates object is a placeholder reference to an actual object and does not contain valid data.
/// </summary>
Expand DownExpand Up@@ -373,7 +362,7 @@ public OpenApiSchema(OpenApiSchema schema)
MinLength = schema?.MinLength ?? MinLength;
Pattern = schema?.Pattern ?? Pattern;
MultipleOf = schema?.MultipleOf ?? MultipleOf;
_default = schema?.Default != null ? JsonNodeCloneHelper.Clone(schema?.Default) : null;
Default = schema?.Default != null ? JsonNodeCloneHelper.Clone(schema?.Default) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
ReadOnly = schema?.ReadOnly ?? ReadOnly;
WriteOnly = schema?.WriteOnly ?? WriteOnly;
AllOf = schema?.AllOf != null ? new List<OpenApiSchema>(schema.AllOf) : null;
Expand All@@ -392,8 +381,8 @@ public OpenApiSchema(OpenApiSchema schema)
AdditionalPropertiesAllowed = schema?.AdditionalPropertiesAllowed ?? AdditionalPropertiesAllowed;
AdditionalProperties = schema?.AdditionalProperties != null ? new(schema?.AdditionalProperties) : null;
Discriminator = schema?.Discriminator != null ? new(schema?.Discriminator) : null;
_example = schema?.Example != null ? JsonNodeCloneHelper.Clone(schema?.Example) : null;
_examples = schema?.Examples != null ? new List<JsonNode>(schema.Examples) : null;
Example = schema?.Example != null ? JsonNodeCloneHelper.Clone(schema?.Example) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
Examples = schema?.Examples != null ? new List<JsonNode>(schema.Examples) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
Enum = schema?.Enum != null ? new List<JsonNode>(schema.Enum) : null;
Nullable = schema?.Nullable ?? Nullable;
ExternalDocs = schema?.ExternalDocs != null ? new(schema?.ExternalDocs) : null;
Expand All@@ -403,6 +392,7 @@ public OpenApiSchema(OpenApiSchema schema)
UnresolvedReference = schema?.UnresolvedReference ?? UnresolvedReference;
Reference = schema?.Reference != null ? new(schema?.Reference) : null;
Annotations = schema?.Annotations != null ? new Dictionary<string, object>(schema?.Annotations) : null;
UnrecognizedKeywords = schema?.UnrecognizedKeywords != null ? new Dictionary<string, JsonNode>(schema?.UnrecognizedKeywords) : null;

Check warning

Code scanning / CodeQL

Virtual call in constructor or destructor

Avoid virtual calls in a constructor or destructor.
}

/// <summary>
Expand DownExpand Up@@ -430,7 +420,7 @@ public void SerializeInternal(IOpenApiWriter writer, OpenApiSpecVersion version,

if (version == OpenApiSpecVersion.OpenApi3_1)
{
WriteV31Properties(writer);
WriteJsonSchemaKeywords(writer);
}

// title
Expand DownExpand Up@@ -554,6 +544,12 @@ public void SerializeInternal(IOpenApiWriter writer, OpenApiSpecVersion version,
// extensions
writer.WriteExtensions(Extensions, version);

// Unrecognized keywords
if (UnrecognizedKeywords.Any())
{
writer.WriteOptionalMap(OpenApiConstants.UnrecognizedKeywords, UnrecognizedKeywords, (w,s) => w.WriteAny(s));
}

writer.WriteEndObject();
}

Expand All@@ -564,7 +560,7 @@ public virtual void SerializeAsV2(IOpenApiWriter writer)
SerializeAsV2(writer: writer, parentRequiredProperties: new HashSet<string>(), propertyName: null);
}

internal void WriteV31Properties(IOpenApiWriter writer)
internal void WriteJsonSchemaKeywords(IOpenApiWriter writer)
{
writer.WriteProperty(OpenApiConstants.Id, Id);
writer.WriteProperty(OpenApiConstants.DollarSchema, Schema);
Expand All@@ -577,7 +573,7 @@ internal void WriteV31Properties(IOpenApiWriter writer)
writer.WriteProperty(OpenApiConstants.V31ExclusiveMaximum, V31ExclusiveMaximum);
writer.WriteProperty(OpenApiConstants.V31ExclusiveMinimum, V31ExclusiveMinimum);
writer.WriteProperty(OpenApiConstants.UnevaluatedProperties, UnevaluatedProperties, false);
writer.WriteOptionalCollection(OpenApiConstants.Examples, _examples, (nodeWriter, s) => nodeWriter.WriteAny(s));
writer.WriteOptionalCollection(OpenApiConstants.Examples, Examples, (nodeWriter, s) => nodeWriter.WriteAny(s));
writer.WriteOptionalMap(OpenApiConstants.PatternProperties, PatternProperties, (w, s) => s.SerializeAsV31(w));
}

Expand DownExpand Up@@ -820,15 +816,9 @@ private void SerializeTypeProperty(JsonSchemaType? type, IOpenApiWriter writer,
}
else
{
var list = new List<JsonSchemaType>();
foreach (JsonSchemaType flag in jsonSchemaTypeValues)
{
if (type.Value.HasFlag(flag))
{
list.Add(flag);
}
}

var list = (from JsonSchemaType flag in jsonSchemaTypeValues
where type.Value.HasFlag(flag)
select flag).ToList();
writer.WriteOptionalCollection(OpenApiConstants.Type, list, (w, s) => w.WriteValue(s.ToIdentifier()));
}
}
Expand All@@ -850,16 +840,9 @@ private void UpCastSchemaTypeToV31(JsonSchemaType? type, IOpenApiWriter writer)
{
// create a new array and insert the type and "null" as values
Type = type | JsonSchemaType.Null;
var list = new List<string>();
foreach (JsonSchemaType? flag in jsonSchemaTypeValues)
{
// Check if the flag is set in 'type' using a bitwise AND operation
if (Type.Value.HasFlag(flag))
{
list.Add(flag.ToIdentifier());
}
}

var list = (from JsonSchemaType? flag in jsonSchemaTypeValues// Check if the flag is set in 'type' using a bitwise AND operation
where Type.Value.HasFlag(flag)
select flag.ToIdentifier()).ToList();
writer.WriteOptionalCollection(OpenApiConstants.Type, list, (w, s) => w.WriteValue(s));
}

Expand Down
14 changes: 13 additions & 1 deletion src/Microsoft.OpenApi/Reader/V31/OpenApiSchemaDeserializer.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -7,6 +7,8 @@
using Microsoft.OpenApi.Reader.ParseNodes;
using System.Collections.Generic;
using System.Globalization;
using System.Linq;
using System.Text.Json.Nodes;

namespace Microsoft.OpenApi.Reader.V31
{
Expand DownExpand Up@@ -254,7 +256,17 @@ public static OpenApiSchema LoadSchema(ParseNode node, OpenApiDocument hostDocum

foreach (var propertyNode in mapNode)
{
propertyNode.ParseField(schema, _openApiSchemaFixedFields, _openApiSchemaPatternFields);
bool isRecognized = _openApiSchemaFixedFields.ContainsKey(propertyNode.Name) ||
_openApiSchemaPatternFields.Any(p => p.Key(propertyNode.Name));
Comment thread
baywet marked this conversation as resolved.

if (isRecognized)
{
propertyNode.ParseField(schema, _openApiSchemaFixedFields, _openApiSchemaPatternFields);
}
else
{
schema.UnrecognizedKeywords[propertyNode.Name] = propertyNode.JsonNode;
}
}

if (schema.Extensions.ContainsKey(OpenApiConstants.NullableExtension))
Expand Down
20 changes: 20 additions & 0 deletions src/Microsoft.OpenApi/Writers/OpenApiWriterExtensions.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -5,6 +5,7 @@
using System.Collections;
using System.Collections.Generic;
using System.Linq;
using System.Text.Json.Nodes;
using Microsoft.OpenApi.Interfaces;

namespace Microsoft.OpenApi.Writers
Expand DownExpand Up@@ -253,6 +254,25 @@ public static void WriteRequiredMap(
writer.WriteMapInternal(name, elements, action);
}

/// <summary>
/// Write the optional Open API element map (string to string mapping).
/// </summary>
/// <param name="writer">The Open API writer.</param>
/// <param name="name">The property name.</param>
/// <param name="elements">The map values.</param>
/// <param name="action">The map element writer action.</param>
public static void WriteOptionalMap(
this IOpenApiWriter writer,
string name,
IDictionary<string, JsonNode> elements,
Action<IOpenApiWriter, JsonNode> action)
{
if (elements != null && elements.Any())
{
writer.WriteMapInternal(name, elements, action);
}
}

/// <summary>
/// Write the optional Open API element map (string to string mapping).
/// </summary>
Expand Down
Original file line numberDiff line numberDiff line change
Expand Up@@ -495,5 +495,21 @@ public void ParseSchemaWithConstWorks()
var schemaString = writer.ToString();
schemaString.MakeLineBreaksEnvironmentNeutral().Should().Be(expected.MakeLineBreaksEnvironmentNeutral());
}

[Fact]
public void ParseSchemaWithUnrecognizedKeywordsWorks()
{
var input = @"{
""type"": ""string"",
""format"": ""date-time"",
""customKeyword"": ""customValue"",
""anotherKeyword"": 42,
""x-test"": ""test""
}
";
var schema = OpenApiModelFactory.Parse<OpenApiSchema>(input, OpenApiSpecVersion.OpenApi3_1, out _, "json");
schema.UnrecognizedKeywords.Should().HaveCount(2);
}

}
}
Original file line numberDiff line numberDiff line change
Expand Up@@ -28,14 +28,3 @@ required:
- name

$dynamicAnchor: "addressDef"
definitions:
address:
$dynamicAnchor: "addressDef"
type: "object"
properties:
street:
type: "string"
city:
type: "string"
postalCode:
type: "string"
41 changes: 34 additions & 7 deletions test/Microsoft.OpenApi.Tests/Models/OpenApiSchemaTests.cs
Original file line numberDiff line numberDiff line change
Expand Up@@ -22,7 +22,7 @@ namespace Microsoft.OpenApi.Tests.Models
[Collection("DefaultSettings")]
public class OpenApiSchemaTests
{
public static OpenApiSchema BasicSchema = new();
private static readonly OpenApiSchema BasicSchema = new();

public static readonly OpenApiSchema AdvancedSchemaNumber = new()
{
Expand DownExpand Up@@ -602,15 +602,42 @@ public void OpenApiWalkerVisitsOpenApiSchemaNot()
// Assert
visitor.Titles.Count.Should().Be(2);
}
}

internal class SchemaVisitor : OpenApiVisitorBase
{
public List<string> Titles = new();
[Fact]
public void SerializeSchemaWithUnrecognizedPropertiesWorks()
{
// Arrange
var schema = new OpenApiSchema
{
UnrecognizedKeywords = new Dictionary<string, JsonNode>()
{
["customKeyWord"] = "bar",
["anotherKeyword"] = 42
}
};

public override void Visit(OpenApiSchema schema)
var expected = @"{
""unrecognizedKeywords"": {
""customKeyWord"": ""bar"",
""anotherKeyword"": 42
}
}";

// Act
var actual = schema.SerializeAsJson(OpenApiSpecVersion.OpenApi3_1);

// Assert
actual.MakeLineBreaksEnvironmentNeutral().Should().Be(expected.MakeLineBreaksEnvironmentNeutral());
}

internal class SchemaVisitor : OpenApiVisitorBase
{
Titles.Add(schema.Title);
public List<string> Titles = new();

public override void Visit(OpenApiSchema schema)
{
Titles.Add(schema.Title);
}
}
}
}
Original file line numberDiff line numberDiff line change
Expand Up@@ -512,6 +512,7 @@ namespace Microsoft.OpenApi.Models
public const string Type = "type";
public const string UnevaluatedProperties = "unevaluatedProperties";
public const string UniqueItems = "uniqueItems";
public const string UnrecognizedKeywords = "unrecognizedKeywords";
public const string Url = "url";
public const string V2ReferenceUri = "https://registry/definitions/";
public const string V31ExclusiveMaximum = "exclusiveMaximum";
Expand DownExpand Up@@ -925,6 +926,7 @@ namespace Microsoft.OpenApi.Models
public virtual bool UnEvaluatedProperties { get; set; }
public virtual bool UnevaluatedProperties { get; set; }
public virtual bool? UniqueItems { get; set; }
public virtual System.Collections.Generic.IDictionary<string, System.Text.Json.Nodes.JsonNode> UnrecognizedKeywords { get; set; }
public virtual bool UnresolvedReference { get; set; }
public virtual decimal? V31ExclusiveMaximum { get; set; }
public virtual decimal? V31ExclusiveMinimum { get; set; }
Expand DownExpand Up@@ -1859,6 +1861,7 @@ namespace Microsoft.OpenApi.Writers
public static void WriteOptionalCollection<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IEnumerable<T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, T> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, bool> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, bool> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, string> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, string> action) { }
public static void WriteOptionalMap(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, System.Text.Json.Nodes.JsonNode> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, System.Text.Json.Nodes.JsonNode> action) { }
public static void WriteOptionalMap<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, T> action)
where T : Microsoft.OpenApi.Interfaces.IOpenApiElement { }
public static void WriteOptionalMap<T>(this Microsoft.OpenApi.Writers.IOpenApiWriter writer, string name, System.Collections.Generic.IDictionary<string, T> elements, System.Action<Microsoft.OpenApi.Writers.IOpenApiWriter, string, T> action)
Expand Down