Skip to content

Fix CI build failures: remove unused log message and add missing DPoP API entries#3551

Merged
MZOLN merged 2 commits into
dev8xfrom
iinglese/remove-idx10618
Jul 16, 2026
Merged

Fix CI build failures: remove unused log message and add missing DPoP API entries#3551
MZOLN merged 2 commits into
dev8xfrom
iinglese/remove-idx10618

Conversation

@iNinja

@iNinja iNinja commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Changes

Remove IDX10618 log message

IDX10618 is no longer referenced anywhere in source. The constant in LogMessages.cs is commented out (following the existing pattern) and its entry removed from InternalAPI.Shipped.txt, fixing the VerifyResourceUsage script failure.

Add missing DPoP API entries to Shipped.txt files

Several public and internal symbols from Microsoft.IdentityModel.Dpop were present in the 8.19.2 release but absent from the per-TFM PublicAPI.Shipped.txt and InternalAPI.Shipped.txt files, causing RS0016/RS0051 errors. These entries have been added to all relevant TFM-specific API files (net462, net472, net6.0, net8.0, net9.0, net10.0, netstandard2.0), with DpopValidationOptions.TimeProvider restricted to net8.0+ as it is guarded by #if NET8_0_OR_GREATER.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3267f260-239f-4520-91f4-49cf589cf6e7
@iNinja
iNinja requested a review from a team as a code owner July 16, 2026 12:30
Symbols shipped in 8.19.2 were absent from the per-TFM PublicAPI.Shipped.txt
and InternalAPI.Shipped.txt files, causing RS0016/RS0051 build failures.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 3267f260-239f-4520-91f4-49cf589cf6e7
@iNinja iNinja changed the title Remove IDX10618 log message - no longer used Fix CI build failures: remove unused log message and add missing DPoP API entries Jul 16, 2026
@MZOLN
MZOLN merged commit d3a7ddb into dev8x Jul 16, 2026
2 checks passed
This was referenced Jul 22, 2026
This was referenced Jul 23, 2026
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