Skip to content

Update Microsoft.OpenApi packages version to 3.10.2 - #1504

Open
johnthcall wants to merge 2 commits into
wiremock:masterfrom
johnthcall:patch-1
Open

Update Microsoft.OpenApi packages version to 3.10.2#1504
johnthcall wants to merge 2 commits into
wiremock:masterfrom
johnthcall:patch-1

Conversation

@johnthcall

@johnthcall johnthcall commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

References

Upgrading Microsoft.OpenApi packages to a version which takes SharpYalm 2.1.5 with the following fix xoofx/SharpYaml#130

Submitter checklist

  • Recommended: Join WireMock Slack to get any help in #help-contributing or a project-specific channel like #wiremock-java
  • The PR request is well described and justified, including the body and the references
  • The PR title represents the desired changelog entry
  • The repository's code style is followed (see the contributing guide)
  • Test coverage that demonstrates that the change works as expected
  • For new features, there's necessary documentation in this pull request or in a subsequent PR to wiremock.org

@StefH StefH added the dependencies Pull requests that update a dependency file label Sep 1, 2026
Comment thread src/WireMock.Net.OpenApiParser/WireMock.Net.OpenApiParser.csproj Outdated
@johnthcall johnthcall changed the title Update SharpYaml package version to 2.1.5 Update Microsoft.OpenApi packages version to 3.10.2 Sep 1, 2026
@johnthcall

Copy link
Copy Markdown
Contributor Author

@StefH the following change was introduced in the latest Microsoft.OpenApi.YamlReader microsoft/OpenAPI.NET#3027 which added limits MaxAliasExpansionNodeCount. This can be overridden when creating OpenApiYamlReader in the constructor of WireMockOpenApiParser. WireMockOpenApiParserTests.FromText_UsingYaml_ShouldReturnMappings requires increasing the AliasExpansionNodeCount from 5k to over 100k. Should WireMockOpenApiParser take a nullable OpenApiYamlReaderSettings or integer MaxAliasExpansionNodeCount to allow customers to increase this? Or should I revert the PR to just be the SharpYaml version increase?

@StefH

StefH commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Yes please keep it to sharpyml.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

StackOverflowException with deeply nested flow sequences/mappings

2 participants