[PR #346] [MERGED] Fix recorded requests skipped by request logger #868

Closed
opened 2025-12-29 08:33:50 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/wiremock/WireMock.Net/pull/346
Author: @vitaliydavydiak
Created: 9/16/2019
Status: Merged
Merged: 9/17/2019
Merged by: @StefH

Base: masterHead: log-recorded-requests


📝 Commits (6)

  • 6c170fb Fix recorded requests skipped request logger.
  • 4b8291e Add unit test
  • 4786d12 Use different solution
  • a55ef53 Introduce IsRecordedByProxy property on Mapping class
  • 94ccb8d Cleanup empty lines
  • 56fb16e Refactored fix suggested way

📊 Changes

4 files changed (+43 additions, -1 deletions)

View changed files

📝 src/WireMock.Net/IMapping.cs (+8 -0)
📝 src/WireMock.Net/Mapping.cs (+3 -0)
📝 src/WireMock.Net/Owin/WireMockMiddleware.cs (+2 -1)
📝 test/WireMock.Net.Tests/FluentMockServerTests.Proxy.cs (+30 -0)

📄 Description

When proxy is enabled the recorded requests are mistaken (IMO) for admin interface requests and skipped in LogEntries. Actually none of requests logged in proxy mode. It is valuable to monitor that while recording.


🔄 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/346 **Author:** [@vitaliydavydiak](https://github.com/vitaliydavydiak) **Created:** 9/16/2019 **Status:** ✅ Merged **Merged:** 9/17/2019 **Merged by:** [@StefH](https://github.com/StefH) **Base:** `master` ← **Head:** `log-recorded-requests` --- ### 📝 Commits (6) - [`6c170fb`](https://github.com/wiremock/WireMock.Net/commit/6c170fb57db423974cab4a2fa8e045b71e19bff2) Fix recorded requests skipped request logger. - [`4b8291e`](https://github.com/wiremock/WireMock.Net/commit/4b8291e95a98b967f7644ad2487f11d6124705f4) Add unit test - [`4786d12`](https://github.com/wiremock/WireMock.Net/commit/4786d123ecddd87f5d58383c383fff0a2f913b06) Use different solution - [`a55ef53`](https://github.com/wiremock/WireMock.Net/commit/a55ef5336dc71d1e1275696410efcedb605b6736) Introduce IsRecordedByProxy property on Mapping class - [`94ccb8d`](https://github.com/wiremock/WireMock.Net/commit/94ccb8dcd754265323dd47408f237d2ca313bf63) Cleanup empty lines - [`56fb16e`](https://github.com/wiremock/WireMock.Net/commit/56fb16e5ba652858797d796a001a54105abe8101) Refactored fix suggested way ### 📊 Changes **4 files changed** (+43 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `src/WireMock.Net/IMapping.cs` (+8 -0) 📝 `src/WireMock.Net/Mapping.cs` (+3 -0) 📝 `src/WireMock.Net/Owin/WireMockMiddleware.cs` (+2 -1) 📝 `test/WireMock.Net.Tests/FluentMockServerTests.Proxy.cs` (+30 -0) </details> ### 📄 Description When proxy is enabled the recorded requests are mistaken (IMO) for admin interface requests and skipped in LogEntries. Actually none of requests logged in proxy mode. It is valuable to monitor that while recording. --- <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 08:33:50 +01:00
adam closed this issue 2025-12-29 08:33:50 +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#868