Releases Hidi

.github/workflows/docker.yml

@@ -30,13 +30,13 @@ jobs:
         id: getversion
       - name: Push to GitHub Packages - Nightly
         if: ${{ github.ref == 'refs/heads/vnext' }}
-        uses: docker/build-push-action@v5.4.0
+        uses: docker/build-push-action@v6.1.0
         with:
           push: true
           tags: ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}:nightly
       - name: Push to GitHub Packages - Release
         if: ${{ github.ref == 'refs/heads/master' }}
-        uses: docker/build-push-action@v5.4.0
+        uses: docker/build-push-action@v6.1.0
         with:
           push: true
           tags: ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}:latest,${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}:${{ steps.getversion.outputs.version }}

src/Microsoft.OpenApi.Hidi/Microsoft.OpenApi.Hidi.csproj

@@ -9,7 +9,7 @@
     <Nullable>enable</Nullable>
     <ToolCommandName>hidi</ToolCommandName>
     <PackageOutputPath>./../../artifacts</PackageOutputPath>
-    <Version>1.4.5</Version>
+    <Version>1.4.6</Version>
     <Description>OpenAPI.NET CLI tool for slicing OpenAPI documents</Description>
     <SignAssembly>true</SignAssembly>
     <!-- https://github.com/dotnet/sourcelink/blob/main/docs/README.md#embeduntrackedsources -->
@@ -35,7 +35,7 @@
     <PackageReference Include="Microsoft.Extensions.Logging.Debug" Version="8.0.0" />
     <PackageReference Include="System.CommandLine" Version="2.0.0-beta4.22272.1" />
     <PackageReference Include="Microsoft.OData.Edm" Version="7.21.3" />
-    <PackageReference Include="Microsoft.OpenApi.OData" Version="1.6.6" />
+    <PackageReference Include="Microsoft.OpenApi.OData" Version="1.6.7" />
     <PackageReference Include="Microsoft.OpenApi.ApiManifest" Version="0.5.0-preview" />
     <PackageReference Include="System.CommandLine.Hosting" Version="0.4.0-alpha.22272.1" />
   </ItemGroup>

src/Microsoft.OpenApi.Readers/V2/OpenApiV2Deserializer.cs

@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft Corporation. All rights reserved.
+// Copyright (c) Microsoft Corporation. All rights reserved.
 // Licensed under the MIT license.
 
 using System.Collections.Generic;
@@ -113,16 +113,16 @@
 
         private static IOpenApiExtension LoadExtension(string name, ParseNode node)
         {
-            if (node.Context.ExtensionParsers.TryGetValue(name, out var parser))
+            if (node.Context.ExtensionParsers.TryGetValue(name, out var parser) && parser(
+                   OpenApiAnyConverter.GetSpecificOpenApiAny(node.CreateAny()),
+                   OpenApiSpecVersion.OpenApi2_0) is { } result)
             {
-                return parser(
-                    OpenApiAnyConverter.GetSpecificOpenApiAny(node.CreateAny()),
-                    OpenApiSpecVersion.OpenApi2_0);
+                return result;
             }
             else
             {
                 return OpenApiAnyConverter.GetSpecificOpenApiAny(node.CreateAny());
             }
         }
 
         private static string LoadString(ParseNode node)

src/Microsoft.OpenApi.Readers/V3/OpenApiV3Deserializer.cs

@@ -171,16 +171,16 @@
 
         private static IOpenApiExtension LoadExtension(string name, ParseNode node)
         {
-            if (node.Context.ExtensionParsers.TryGetValue(name, out var parser))
+            if (node.Context.ExtensionParsers.TryGetValue(name, out var parser) && parser(
+                   OpenApiAnyConverter.GetSpecificOpenApiAny(node.CreateAny()),
+                   OpenApiSpecVersion.OpenApi3_0) is { } result)
             {
-                return parser(
-                    OpenApiAnyConverter.GetSpecificOpenApiAny(node.CreateAny()),
-                    OpenApiSpecVersion.OpenApi3_0);
+                return result;
             }
             else
             {
                 return OpenApiAnyConverter.GetSpecificOpenApiAny(node.CreateAny());
             }
         }
 
         private static string LoadString(ParseNode node)

src/Microsoft.OpenApi/MicrosoftExtensions/OpenApiDeprecationExtension.cs

@@ -78,7 +78,7 @@ public void Write(IOpenApiWriter writer, OpenApiSpecVersion specVersion)
     /// <exception cref="ArgumentOutOfRangeException">When the source element is not an object</exception>
     public static OpenApiDeprecationExtension Parse(IOpenApiAny source)
     {
-        if (source is not OpenApiObject rawObject) throw new ArgumentOutOfRangeException(nameof(source));
+        if (source is not OpenApiObject rawObject) return null;
         var extension = new OpenApiDeprecationExtension();
         if (rawObject.TryGetValue(nameof(RemovalDate).ToFirstCharacterLowerCase(), out var removalDate) && removalDate is OpenApiDateTime removalDateValue)
             extension.RemovalDate = removalDateValue.Value;

src/Microsoft.OpenApi/MicrosoftExtensions/OpenApiEnumFlagsExtension.cs

@@ -45,7 +45,7 @@ public void Write(IOpenApiWriter writer, OpenApiSpecVersion specVersion)
     /// <exception cref="ArgumentOutOfRangeException">When the source element is not an object</exception>
     public static OpenApiEnumFlagsExtension Parse(IOpenApiAny source)
     {
-        if (source is not OpenApiObject rawObject) throw new ArgumentOutOfRangeException(nameof(source));
+        if (source is not OpenApiObject rawObject) return null;
         var extension = new OpenApiEnumFlagsExtension();
         if (rawObject.TryGetValue(nameof(IsFlags).ToFirstCharacterLowerCase(), out var flagsValue) && flagsValue is OpenApiBoolean isFlags)
         {

src/Microsoft.OpenApi/MicrosoftExtensions/OpenApiEnumValuesDescriptionExtension.cs

@@ -64,7 +64,7 @@ public void Write(IOpenApiWriter writer, OpenApiSpecVersion specVersion)
     /// <exception cref="ArgumentOutOfRangeException">When the source element is not an object</exception>
     public static OpenApiEnumValuesDescriptionExtension Parse(IOpenApiAny source)
     {
-        if (source is not OpenApiObject rawObject) throw new ArgumentOutOfRangeException(nameof(source));
+        if (source is not OpenApiObject rawObject) return null;
         var extension = new OpenApiEnumValuesDescriptionExtension();
         if (rawObject.TryGetValue("values", out var values) && values is OpenApiArray valuesArray)
         {

src/Microsoft.OpenApi/MicrosoftExtensions/OpenApiPagingExtension.cs

@@ -73,7 +73,7 @@ public void Write(IOpenApiWriter writer, OpenApiSpecVersion specVersion)
     /// <exception cref="ArgumentOutOfRangeException">When the source element is not an object</exception>
     public static OpenApiPagingExtension Parse(IOpenApiAny source)
     {
-        if (source is not OpenApiObject rawObject) throw new ArgumentOutOfRangeException(nameof(source));
+        if (source is not OpenApiObject rawObject) return null;
         var extension = new OpenApiPagingExtension();
         if (rawObject.TryGetValue(nameof(NextLinkName).ToFirstCharacterLowerCase(), out var nextLinkName) && nextLinkName is OpenApiString nextLinkNameStr)
         {

src/Microsoft.OpenApi/MicrosoftExtensions/OpenApiPrimaryErrorMessageExtension.cs

@@ -1,4 +1,4 @@
-// ------------------------------------------------------------
+// ------------------------------------------------------------
 //  Copyright (c) Microsoft Corporation.  All rights reserved.
 //  Licensed under the MIT License (MIT). See LICENSE in the repo root for license information.
 // ------------------------------------------------------------
@@ -39,7 +39,7 @@ public void Write(IOpenApiWriter writer, OpenApiSpecVersion specVersion)
     /// <returns>The <see cref="OpenApiPrimaryErrorMessageExtension"/>.</returns>
     public static OpenApiPrimaryErrorMessageExtension Parse(IOpenApiAny source)
     {
-        if (source is not OpenApiBoolean rawObject) throw new ArgumentOutOfRangeException(nameof(source));
+        if (source is not OpenApiBoolean rawObject) return null;
         return new()
         {
             IsPrimaryErrorMessage = rawObject.Value

src/Microsoft.OpenApi/MicrosoftExtensions/OpenApiReservedParameterExtension.cs

@@ -41,7 +41,7 @@ public bool? IsReserved
     /// <returns></returns>
     public static OpenApiReservedParameterExtension Parse(IOpenApiAny source)
     {
-        if (source is not OpenApiBoolean rawBoolean) throw new ArgumentOutOfRangeException(nameof(source));
+        if (source is not OpenApiBoolean rawBoolean) return null;
         return new()
         {
             IsReserved = rawBoolean.Value

src/Microsoft.OpenApi/Properties/SRResource.resx

@@ -139,7 +139,7 @@
     <value>Invalid Reference identifier '{0}'.</value>
   </data>
   <data name="InvalidReferenceType" xml:space="preserve">
-    <value>Invalid Reference Type.</value>
+    <value>Invalid Reference Type '{0}'.</value>
   </data>
   <data name="LocalReferenceRequiresType" xml:space="preserve">
     <value>Local reference must have type specified.</value>

src/Microsoft.OpenApi/Services/OpenApiReferenceResolver.cs

@@ -259,6 +259,13 @@
             {
                 try
                 {
+                    var referencedObject = typeof(T).Name;
+                    var referenceType = reference?.Type.ToString();
+                    if (referenceType is not null && !referencedObject.Contains(referenceType))
+                    {
+                        throw new OpenApiException(string.Format(Properties.SRResource.InvalidReferenceType, referenceType));
+                    }
+
                     return _currentDocument.ResolveReference(reference, false) as T;
                 }
                 catch (OpenApiException ex)
@@ -269,7 +276,7 @@
             }
             // The concept of merging references with their target at load time is going away in the next major version
             // External references will not support this approach.
             //else if (_resolveRemoteReferences == true)
             //{
             //    if (_currentDocument.Workspace == null)
             //    {
@@ -283,7 +290,7 @@
             //    }
             //    var target = _currentDocument.Workspace.ResolveReference(reference);
 
             //    // TODO:  If it is a document fragment, then we should resolve it within the current context
 
             //    return target as T;
             //}

test/Microsoft.OpenApi.Readers.Tests/V3Tests/OpenApiDocumentTests.cs

@@ -9,8 +9,10 @@
 using System.Threading;
 using FluentAssertions;
 using Microsoft.OpenApi.Any;
+using Microsoft.OpenApi.Exceptions;
 using Microsoft.OpenApi.Interfaces;
 using Microsoft.OpenApi.Models;
+using Microsoft.OpenApi.Readers.Interface;
 using Microsoft.OpenApi.Validations;
 using Microsoft.OpenApi.Validations.Rules;
 using Microsoft.OpenApi.Writers;
@@ -1355,5 +1357,15 @@ public void ValidateExampleShouldNotHaveDataTypeMismatch()
             var warnings = diagnostic.Warnings;
             Assert.False(warnings.Any());
         }
+
+        [Fact]
+        public void ParseDocumetWithWrongReferenceTypeShouldReturnADiagnosticError()
+        {
+            using var stream = Resources.GetStream(Path.Combine(SampleFolderPath, "docWithWrongRef.json"));
+            _ = new OpenApiStreamReader().Read(stream, out var diagnostic);
+
+            diagnostic.Errors.Should().BeEquivalentTo(new List<OpenApiError> {
+                new( new OpenApiException("Invalid Reference Type 'Schema'.")) });
+        }
     }
 }

test/Microsoft.OpenApi.Readers.Tests/V3Tests/Samples/OpenApiDocument/docWithWrongRef.json

@@ -0,0 +1,27 @@
+{
+    "openapi":"3.0.0",
+    "info":{
+        "title":"some api",
+        "description":"some description",
+        "version": "1"
+    },
+    "servers":[{"url":"https://localhost"}],
+    "paths":{
+        "/count":{
+            "get":{
+                "responses":{
+                    "200":{
+                        "$ref":"#/components/schemas/count"
+                    },
+                },
+            }
+        }
+    },
+    "components":{
+        "schemas":{
+            "count":{
+                "type": "number"
+            }
+        }
+    }
+}

test/Microsoft.OpenApi.Tests/Microsoft.OpenApi.Tests.csproj

@@ -15,7 +15,7 @@
     <PackageReference Include="Moq" Version="4.20.70" />
     <PackageReference Include="Newtonsoft.Json" Version="13.0.3" />
     <PackageReference Include="SharpYaml" Version="2.1.1" />
-    <PackageReference Include="Verify.Xunit" Version="25.0.1" />
+    <PackageReference Include="Verify.Xunit" Version="25.0.4" />
     <PackageReference Include="xunit" Version="2.8.1" />
     <PackageReference Include="xunit.runner.visualstudio" Version="2.8.1" PrivateAssets="all" />
     <PackageReference Include="Microsoft.CSharp" Version="4.7.0" />

test/Microsoft.OpenApi.Tests/MicrosoftExtensions/OpenApiReservedParameterExtensionTests.cs

@@ -17,6 +17,19 @@ public void Parses()
         Assert.NotNull(value);
         Assert.True(value.IsReserved);
     }
+
+    [Fact]
+    public void DoesNotThrowExceptionIfValueIsNull()
+    {
+        var oaiValue = new OpenApiObject
+        {
+            ["foo"] = new OpenApiString("foo")
+        };
+
+        var value = OpenApiReservedParameterExtension.Parse(oaiValue);
+        Assert.Null(value);
+    }
+
     [Fact]
     public void Serializes()
     {