[PR #986] [MERGED] Write logging in case a Matcher throws an exception #1180

Closed
opened 2025-12-29 09:18:48 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/wiremock/WireMock.Net/pull/986
Author: @StefH
Created: 8/3/2023
Status: Merged
Merged: 8/21/2023
Merged by: @StefH

Base: masterHead: stef-404


📝 Commits (10+)

📊 Changes

88 files changed (+1282 additions, -1260 deletions)

View changed files

📝 src/WireMock.Net.Abstractions/Admin/Mappings/StatusModel.cs (+19 -15)
📝 src/WireMock.Net.Abstractions/Admin/Settings/SettingsModel.cs (+0 -5)
📝 src/WireMock.Net.Abstractions/Handlers/IFileSystemHandler.cs (+114 -115)
📝 src/WireMock.Net.Abstractions/Logging/IWireMockLogger.cs (+50 -52)
📝 src/WireMock.Net.Abstractions/Matchers/Request/IRequestMatchResult.cs (+44 -44)
📝 src/WireMock.Net.Abstractions/Matchers/Request/MatchDetail.cs (+19 -14)
📝 src/WireMock.Net.Matchers.CSharpCode/Matchers/CSharpCodeMatcher.cs (+23 -18)
📝 src/WireMock.Net/Authentication/AzureADAuthenticationMatcher.cs (+3 -5)
📝 src/WireMock.Net/Constants/WireMockConstants.cs (+2 -0)
src/WireMock.Net/Extensions/ExceptionExtensions.cs (+16 -0)
📝 src/WireMock.Net/Handlers/LocalFileSystemHandler.cs (+2 -2)
📝 src/WireMock.Net/Mapping.cs (+2 -2)
📝 src/WireMock.Net/Matchers/AbstractJsonPartialMatcher.cs (+7 -13)
📝 src/WireMock.Net/Matchers/ContentTypeMatcher.cs (+6 -8)
📝 src/WireMock.Net/Matchers/ExactMatcher.cs (+9 -15)
📝 src/WireMock.Net/Matchers/ExactObjectMatcher.cs (+5 -10)
📝 src/WireMock.Net/Matchers/GraphQLMatcher.cs (+42 -36)
📝 src/WireMock.Net/Matchers/Helpers/BodyDataMatchScoreCalculator.cs (+3 -13)
📝 src/WireMock.Net/Matchers/IMatcher.cs (+0 -5)
📝 src/WireMock.Net/Matchers/IObjectMatcher.cs (+2 -2)

...and 68 more files

📄 Description

No description provided


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/wiremock/WireMock.Net/pull/986 **Author:** [@StefH](https://github.com/StefH) **Created:** 8/3/2023 **Status:** ✅ Merged **Merged:** 8/21/2023 **Merged by:** [@StefH](https://github.com/StefH) **Base:** `master` ← **Head:** `stef-404` --- ### 📝 Commits (10+) - [`00009c3`](https://github.com/wiremock/WireMock.Net/commit/00009c383db9945b65fdc50b1a1e04d3a5ea91e9) ThrowException - [`d2baff9`](https://github.com/wiremock/WireMock.Net/commit/d2baff95df781f0e326e137682565381f886b248) ... - [`b87cf16`](https://github.com/wiremock/WireMock.Net/commit/b87cf167bcb41257a0c015261830ded5dad2dc83) . - [`eefe026`](https://github.com/wiremock/WireMock.Net/commit/eefe0264ed8ad22231f4ae33160f92c8cc379d39) Merge branch 'master' into stef-404 - [`0ceab87`](https://github.com/wiremock/WireMock.Net/commit/0ceab872f6e4cda266b8dc0da1b836bdabd928de) ... - [`14d4efa`](https://github.com/wiremock/WireMock.Net/commit/14d4efa3b6768084d5c574f50e8175a6a13e75dd) b - [`ff6def9`](https://github.com/wiremock/WireMock.Net/commit/ff6def995cf56a5bbc9498cfba7e36101f288d91) fix test - [`5e7cca4`](https://github.com/wiremock/WireMock.Net/commit/5e7cca471495f9c9520d75c26d7babdf2855004c) ... - [`f4b7ce0`](https://github.com/wiremock/WireMock.Net/commit/f4b7ce0c5469c531095bba4066bf5808b6c67dce) . - [`b088c0f`](https://github.com/wiremock/WireMock.Net/commit/b088c0fe3b55d5e2ae51e9640b60656daf88f135) sonar ### 📊 Changes **88 files changed** (+1282 additions, -1260 deletions) <details> <summary>View changed files</summary> 📝 `src/WireMock.Net.Abstractions/Admin/Mappings/StatusModel.cs` (+19 -15) 📝 `src/WireMock.Net.Abstractions/Admin/Settings/SettingsModel.cs` (+0 -5) 📝 `src/WireMock.Net.Abstractions/Handlers/IFileSystemHandler.cs` (+114 -115) 📝 `src/WireMock.Net.Abstractions/Logging/IWireMockLogger.cs` (+50 -52) 📝 `src/WireMock.Net.Abstractions/Matchers/Request/IRequestMatchResult.cs` (+44 -44) 📝 `src/WireMock.Net.Abstractions/Matchers/Request/MatchDetail.cs` (+19 -14) 📝 `src/WireMock.Net.Matchers.CSharpCode/Matchers/CSharpCodeMatcher.cs` (+23 -18) 📝 `src/WireMock.Net/Authentication/AzureADAuthenticationMatcher.cs` (+3 -5) 📝 `src/WireMock.Net/Constants/WireMockConstants.cs` (+2 -0) ➕ `src/WireMock.Net/Extensions/ExceptionExtensions.cs` (+16 -0) 📝 `src/WireMock.Net/Handlers/LocalFileSystemHandler.cs` (+2 -2) 📝 `src/WireMock.Net/Mapping.cs` (+2 -2) 📝 `src/WireMock.Net/Matchers/AbstractJsonPartialMatcher.cs` (+7 -13) 📝 `src/WireMock.Net/Matchers/ContentTypeMatcher.cs` (+6 -8) 📝 `src/WireMock.Net/Matchers/ExactMatcher.cs` (+9 -15) 📝 `src/WireMock.Net/Matchers/ExactObjectMatcher.cs` (+5 -10) 📝 `src/WireMock.Net/Matchers/GraphQLMatcher.cs` (+42 -36) 📝 `src/WireMock.Net/Matchers/Helpers/BodyDataMatchScoreCalculator.cs` (+3 -13) 📝 `src/WireMock.Net/Matchers/IMatcher.cs` (+0 -5) 📝 `src/WireMock.Net/Matchers/IObjectMatcher.cs` (+2 -2) _...and 68 more files_ </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2025-12-29 09:18:48 +01:00
adam closed this issue 2025-12-29 09:18:48 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/WireMock.Net#1180