[PR #803] [MERGED] WebHook : UseFireAndForget + Delay #1091

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

📋 Pull Request Information

Original PR: https://github.com/wiremock/WireMock.Net/pull/803
Author: @StefH
Created: 9/5/2022
Status: Merged
Merged: 9/12/2022
Merged by: @StefH

Base: masterHead: stef-webhooks


📝 Commits (9)

  • 7ca68ce UseFireAndForget
  • 29c7575 ...
  • 9f3ed06 delay
  • 33fc5f1 async
  • e27decb updated code accorsing to proposal
  • 7e328a0 Change nuget to package reference for WireMock.Net.Console.Net472.Classic, move the new FireAndForget into the main mapping, out of individual webhook mappings making it all or nothing, update tests, change Middleware to await or not the firing of all webhooks. Update models as needed. (#804)
  • b71b2e9 small update
  • cfa8ca9 Tweak middleware and fix bug in example (#806)
  • fb3943f .ConfigureAwait(false)

📊 Changes

22 files changed (+634 additions, -424 deletions)

View changed files

📝 examples/WireMock.Net.Console.Net452.Classic/MainApp.cs (+35 -0)
📝 examples/WireMock.Net.Console.Net472.Classic/WireMock.Net.Console.Net472.Classic.csproj (+10 -9)
📝 examples/WireMock.Net.Console.Net472.Classic/packages.config (+0 -3)
📝 src/WireMock.Net.Abstractions/Admin/Mappings/MappingModel.cs (+5 -0)
📝 src/WireMock.Net.Abstractions/Admin/Mappings/WebhookModel.cs (+9 -10)
📝 src/WireMock.Net.Abstractions/Admin/Mappings/WebhookRequestModel.cs (+17 -2)
📝 src/WireMock.Net.Abstractions/Models/IWebhookRequest.cs (+54 -40)
📝 src/WireMock.Net/Http/WebhookSender.cs (+43 -13)
📝 src/WireMock.Net/IMapping.cs (+5 -0)
📝 src/WireMock.Net/Mapping.cs (+6 -0)
📝 src/WireMock.Net/Matchers/Request/RequestMessageParamMatcher.cs (+1 -1)
📝 src/WireMock.Net/Models/Webhook.cs (+8 -9)
📝 src/WireMock.Net/Models/WebhookRequest.cs (+29 -21)
📝 src/WireMock.Net/Owin/WireMockMiddleware.cs (+31 -4)
📝 src/WireMock.Net/Proxy/ProxyHelper.cs (+1 -0)
📝 src/WireMock.Net/Serialization/MappingConverter.cs (+2 -1)
📝 src/WireMock.Net/Serialization/WebhookMapper.cs (+9 -3)
📝 src/WireMock.Net/Server/IRespondWithAProvider.cs (+7 -0)
📝 src/WireMock.Net/Server/RespondWithAProvider.cs (+10 -1)
📝 test/WireMock.Net.Tests/Owin/WireMockMiddlewareTests.cs (+2 -2)

...and 2 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/803 **Author:** [@StefH](https://github.com/StefH) **Created:** 9/5/2022 **Status:** ✅ Merged **Merged:** 9/12/2022 **Merged by:** [@StefH](https://github.com/StefH) **Base:** `master` ← **Head:** `stef-webhooks` --- ### 📝 Commits (9) - [`7ca68ce`](https://github.com/wiremock/WireMock.Net/commit/7ca68cee767ddcbdb784cfe5a3ef97361070e708) UseFireAndForget - [`29c7575`](https://github.com/wiremock/WireMock.Net/commit/29c7575fa5e93969771bf10fed3d5b25eee6307c) ... - [`9f3ed06`](https://github.com/wiremock/WireMock.Net/commit/9f3ed0683a2569217c7d6190efab6ba0d36df75c) delay - [`33fc5f1`](https://github.com/wiremock/WireMock.Net/commit/33fc5f1188210a642e8faf32601c89f6af06d423) async - [`e27decb`](https://github.com/wiremock/WireMock.Net/commit/e27decb129ee0afdb14a4def00b4a8a2b98079bc) updated code accorsing to proposal - [`7e328a0`](https://github.com/wiremock/WireMock.Net/commit/7e328a03e95e3ccfcbe7ea145462750cd7a54554) Change nuget to package reference for WireMock.Net.Console.Net472.Classic, move the new FireAndForget into the main mapping, out of individual webhook mappings making it all or nothing, update tests, change Middleware to await or not the firing of all webhooks. Update models as needed. (#804) - [`b71b2e9`](https://github.com/wiremock/WireMock.Net/commit/b71b2e90e933e99e619c2bb0aebdc454c0e33a83) small update - [`cfa8ca9`](https://github.com/wiremock/WireMock.Net/commit/cfa8ca9e17e9ba7699911e283693884f0cfbf781) Tweak middleware and fix bug in example (#806) - [`fb3943f`](https://github.com/wiremock/WireMock.Net/commit/fb3943ff1a72491f4827850fce02eca56c213733) .ConfigureAwait(false) ### 📊 Changes **22 files changed** (+634 additions, -424 deletions) <details> <summary>View changed files</summary> 📝 `examples/WireMock.Net.Console.Net452.Classic/MainApp.cs` (+35 -0) 📝 `examples/WireMock.Net.Console.Net472.Classic/WireMock.Net.Console.Net472.Classic.csproj` (+10 -9) 📝 `examples/WireMock.Net.Console.Net472.Classic/packages.config` (+0 -3) 📝 `src/WireMock.Net.Abstractions/Admin/Mappings/MappingModel.cs` (+5 -0) 📝 `src/WireMock.Net.Abstractions/Admin/Mappings/WebhookModel.cs` (+9 -10) 📝 `src/WireMock.Net.Abstractions/Admin/Mappings/WebhookRequestModel.cs` (+17 -2) 📝 `src/WireMock.Net.Abstractions/Models/IWebhookRequest.cs` (+54 -40) 📝 `src/WireMock.Net/Http/WebhookSender.cs` (+43 -13) 📝 `src/WireMock.Net/IMapping.cs` (+5 -0) 📝 `src/WireMock.Net/Mapping.cs` (+6 -0) 📝 `src/WireMock.Net/Matchers/Request/RequestMessageParamMatcher.cs` (+1 -1) 📝 `src/WireMock.Net/Models/Webhook.cs` (+8 -9) 📝 `src/WireMock.Net/Models/WebhookRequest.cs` (+29 -21) 📝 `src/WireMock.Net/Owin/WireMockMiddleware.cs` (+31 -4) 📝 `src/WireMock.Net/Proxy/ProxyHelper.cs` (+1 -0) 📝 `src/WireMock.Net/Serialization/MappingConverter.cs` (+2 -1) 📝 `src/WireMock.Net/Serialization/WebhookMapper.cs` (+9 -3) 📝 `src/WireMock.Net/Server/IRespondWithAProvider.cs` (+7 -0) 📝 `src/WireMock.Net/Server/RespondWithAProvider.cs` (+10 -1) 📝 `test/WireMock.Net.Tests/Owin/WireMockMiddlewareTests.cs` (+2 -2) _...and 2 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 08:34:38 +01:00
adam closed this issue 2025-12-29 08:34:38 +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#1091