[PR #1278] [MERGED] changed null check in JSONPathMatcher and JmesPathMatcher to ensure t… #1311

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

📋 Pull Request Information

Original PR: https://github.com/wiremock/WireMock.Net/pull/1278
Author: @jollyjoyce1995
Created: 4/16/2025
Status: Merged
Merged: 4/23/2025
Merged by: @StefH

Base: masterHead: master


📝 Commits (1)

  • 94240bd changed null check in JSONPathMatcher and JmesPathMatcher to ensure that the body is not an empty string (the json parse would throw an exception at this point)

📊 Changes

2 files changed (+2 additions, -2 deletions)

View changed files

📝 src/WireMock.Net/Matchers/JSONPathMatcher.cs (+1 -1)
📝 src/WireMock.Net/Matchers/JmesPathMatcher.cs (+1 -1)

📄 Description

Fixes #1277


🔄 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/1278 **Author:** [@jollyjoyce1995](https://github.com/jollyjoyce1995) **Created:** 4/16/2025 **Status:** ✅ Merged **Merged:** 4/23/2025 **Merged by:** [@StefH](https://github.com/StefH) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`94240bd`](https://github.com/wiremock/WireMock.Net/commit/94240bd7bc9cd3999262138f7f62416a96bc2570) changed null check in JSONPathMatcher and JmesPathMatcher to ensure that the body is not an empty string (the json parse would throw an exception at this point) ### 📊 Changes **2 files changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `src/WireMock.Net/Matchers/JSONPathMatcher.cs` (+1 -1) 📝 `src/WireMock.Net/Matchers/JmesPathMatcher.cs` (+1 -1) </details> ### 📄 Description Fixes #1277 --- <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:19:15 +01:00
adam closed this issue 2025-12-29 09:19:15 +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#1311