Skip to content

Update didx509cpp from 0.12.0 to 0.99.0#7943

Merged
achamayou merged 3 commits into
microsoft:mainfrom
achamayou:update_didx509cpp_from_0.12.0_to_0.99.0
Jun 15, 2026
Merged

Update didx509cpp from 0.12.0 to 0.99.0#7943
achamayou merged 3 commits into
microsoft:mainfrom
achamayou:update_didx509cpp_from_0.12.0_to_0.99.0

Conversation

@achamayou

@achamayou achamayou commented Jun 15, 2026

Copy link
Copy Markdown
Member

Updates the vendored didx509cpp header to upstream tag didx509cpp-0.99.0 (commit f797d09cc3dcadca41844f75422650c714718e3c).

  • 3rdparty/exported/didx509cpp/didx509cpp.h: refreshed to the 0.99.0 header (byte-identical to upstream at that tag).
  • cgmanifest.json: repinned the component to didx509cpp-0.99.0 / f797d09.
  • CHANGELOG.md: noted the dependency update under [7.0.5].

Validation

didx509cpp is header-only with no in-tree build/test target, so the update was validated against the upstream microsoft/didx509cpp test suite:

  • GCC 13.2.0: doctest 44 cases / 382 assertions passing.
  • Clang 18 + ASAN/UBSAN/LSAN: 44 / 382 passing.
  • Vendored header blob is byte-identical to upstream at didx509cpp-0.99.0.

Closes #7939

Copilot AI review requested due to automatic review settings June 15, 2026 15:56
@achamayou achamayou requested a review from a team as a code owner June 15, 2026 15:56
@achamayou achamayou added the run-long-test Run Long Test job label Jun 15, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the vendored didx509cpp header to upstream didx509cpp-0.99.0, repins the component metadata, and records the dependency bump in the changelog.

Changes:

  • Refreshed 3rdparty/exported/didx509cpp/didx509cpp.h to the upstream 0.99.0 header.
  • Updated cgmanifest.json to point at tag didx509cpp-0.99.0 / commit f797d09....
  • Added a [7.0.5] changelog entry noting the dependency update.

Custom instructions used:

  • .github/copilot-instructions.md
  • .github/instructions/reviewing.instructions.md
  • .github/instructions/changelog.instructions.md

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
3rdparty/exported/didx509cpp/didx509cpp.h Vendor refresh to upstream didx509cpp-0.99.0.
cgmanifest.json Repins the dependency tag/commit for component governance.
CHANGELOG.md Notes the dependency update under [7.0.5].

Comment thread cgmanifest.json
@achamayou achamayou merged commit 722bbe8 into microsoft:main Jun 15, 2026
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-long-test Run Long Test job

Projects

None yet

Development

Successfully merging this pull request may close these issues.

did509cpp upgrade

3 participants