Regression tests: Added inner exception detail and invalid signature failure due to invalid algorithm used #2811

iNinja · 2024-09-05T14:06:59Z

Regression tests: Added inner exception detail and invalid signature failure due to invalid algorithm used

Added inner exception detail to provide inner exceptions without allocating them ahead of time
Wrap invalid algorithm validation error inside an invalid signature validation error to match the existing path
Added test case comparing the invalid algorithm on the signature validation error

Part of #2711

…cating them ahead of time

…alidation error to match the existing path

…dation error

src/Microsoft.IdentityModel.Tokens/Validation/Results/Details/ExceptionDetail.cs

FuPingFranco

LGTM

iNinja added 3 commits September 5, 2024 15:00

Added inner exception detail to provide inner exceptions without allo…

6e98ec8

…cating them ahead of time

Wrap invalid algorithm validation error inside an invalid signature v…

2cdb7ca

…alidation error to match the existing path

Added test case comparing the invalid algorithm on the signature vali…

170c421

…dation error

iNinja requested a review from a team as a code owner September 5, 2024 14:07

Moved test id to the constructor

eec5b0d

keegan-caruso reviewed Sep 5, 2024

View reviewed changes

src/Microsoft.IdentityModel.Tokens/Validation/Results/Details/ExceptionDetail.cs Show resolved Hide resolved

FuPingFranco approved these changes Sep 5, 2024

View reviewed changes

keegan-caruso approved these changes Sep 5, 2024

View reviewed changes

iNinja merged commit 957ed51 into dev Sep 6, 2024
6 checks passed

iNinja deleted the iinglese/regression-tests-inner-exceptiondetail branch November 15, 2024 12:15