Skip to content

Bump Microsoft.OpenApi.Readers to 1.6.31 to remediate SharpYaml DoS advisory (MVS-2026-hqm8-ggjx) - #3083

Open
Crash Collison (tehcrashxor) wants to merge 2 commits into
mainfrom
fix/openapi-readers-sharpyaml-mvs-2026-hqm8-ggjx
Open

Bump Microsoft.OpenApi.Readers to 1.6.31 to remediate SharpYaml DoS advisory (MVS-2026-hqm8-ggjx)#3083
Crash Collison (tehcrashxor) wants to merge 2 commits into
mainfrom
fix/openapi-readers-sharpyaml-mvs-2026-hqm8-ggjx

Conversation

@tehcrashxor

Copy link
Copy Markdown
Member

Summary

Bumps Microsoft.OpenApi.Readers from 1.6.11 to 1.6.31 to remediate a Component Governance security alert on the transitive dependency SharpYaml.

Filed as CG alert 19024728 against the Power Fx-Official pipeline.

The vulnerability

MVS-2026-hqm8-ggjx (severity: medium) — SharpYaml versions before 3.4.0 are vulnerable to denial-of-service when processing deeply nested YAML. The advisory requires 2.1.5+ on the 2.x line.

SharpYaml is not referenced directly anywhere in this repo. It arrives purely transitively:

Microsoft.PowerFx.Connectors
  └─ Microsoft.OpenApi.Readers 1.6.11
       └─ SharpYaml 2.1.0   ← vulnerable

Why 1.6.31 specifically

Walking the nuspec dependency of each 1.6.x release:

Microsoft.OpenApi.Readers SharpYaml
1.6.11 (current) 2.1.0 ❌
1.6.14 – 1.6.28 2.1.1 ❌
1.6.31 2.1.5

1.6.31 is the first 1.6.x release that satisfies the advisory. Bumping the reader avoids adding a direct SharpYaml pin to the dependency surface of the shipped Microsoft.PowerFx.Connectors package, and stays on the 1.6.x line (2.x is still preview and has breaking API changes).

Changes

  • Microsoft.PowerFx.Connectors.csproj — 1.6.11 → 1.6.31
  • Microsoft.PowerFx.TexlFunctionExporter.csproj — 1.6.11 → 1.6.31

Both were bumped together so the exporter does not silently downgrade the resolved graph.

Validation

  • Full solution restore clean
  • SharpYaml now resolves to 2.1.5 in all three affected projects (Microsoft.PowerFx.Connectors, Microsoft.PowerFx.Connectors.Tests, Microsoft.PowerFx.TexlFunctionExporter), verified via project.assets.json
  • Microsoft.PowerFx.Connectors and Microsoft.PowerFx.TexlFunctionExporter build clean — 0 warnings, 0 errors
  • Microsoft.PowerFx.Connectors.Tests: 549 passed, 0 failed, 20 skipped

Downstream impact

Microsoft.PowerFx.Connectors declares Microsoft.OpenApi.Readers as a nuspec dependency, so consumers inherit this fix once they pick up a build containing it. This also clears the same SharpYaml advisory on the internal bic/power-fx-internal repo (CG alert 19025149), which consumes Microsoft.PowerFx.Connectors as a package rather than referencing SharpYaml itself.

…dvisory

Microsoft.OpenApi.Readers 1.6.11 has a transitive dependency on
SharpYaml 2.1.0, which is vulnerable to a denial-of-service attack when
processing deeply nested YAML (MVS-2026-hqm8-ggjx). The advisory requires
SharpYaml 2.1.5 or later on the 2.x line.

Microsoft.OpenApi.Readers 1.6.31 is the first 1.6.x release whose nuspec
depends on SharpYaml 2.1.5, so bumping the reader clears the advisory
without introducing a direct SharpYaml pin into the dependency surface of
the shipped Microsoft.PowerFx.Connectors package.

SharpYaml is not referenced directly anywhere in the repo; it arrives only
through this package. After the bump, SharpYaml resolves to 2.1.5 in
Microsoft.PowerFx.Connectors, Microsoft.PowerFx.Connectors.Tests, and
Microsoft.PowerFx.TexlFunctionExporter.

Verified with a full solution restore, a clean build of the affected
projects, and Microsoft.PowerFx.Connectors.Tests (549 passed, 0 failed,
20 skipped).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@tehcrashxor
Crash Collison (tehcrashxor) requested a review from a team as a code owner August 21, 2026 23:30
@jas-valgotar

Copy link
Copy Markdown
Contributor

✅ No public API change.

@jas-valgotar

Copy link
Copy Markdown
Contributor

✅ No public API change.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants