[PR #32] [MERGED] [Feature] Add support for client certificate password and test with real services that require client certificate auth #739

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

📋 Pull Request Information

Original PR: https://github.com/wiremock/WireMock.Net/pull/32
Author: @phillee007
Created: 6/10/2017
Status: Merged
Merged: 6/16/2017
Merged by: @StefH

Base: masterHead: master


📝 Commits (10+)

  • 98eed7f Add support for client certificate password and test with some real services that require client certificates. Also update so when proxying, the path and query will be passed to the proxied api.
  • 3c26f44 Add correct param
  • f3a11fa Update to read certificate from store instead of file
  • 4d9e928 Merge branch 'master' of https://github.com/StefH/WireMock.Net
  • 3358e8e Add support for client certificate password and test with some real services that require client certificates. Also update so when proxying, the path and query will be passed to the proxied api.
  • 70668b2 Add correct param
  • 7dc46a1 Merge branch 'master' of https://github.com/phillee007/WireMock.Net
  • 699156a Fixup PR issues
  • 753522f Merge branch 'master' of https://github.com/phillee007/WireMock.Net
  • 1094af7 Add support for client certificate password and test with some real services that require client certificates. Also update so when proxying, the path and query will be passed to the proxied api.

📊 Changes

11 files changed (+148 additions, -46 deletions)

View changed files

📝 examples/WireMock.Net.Console.Record.NETCoreApp/Program.cs (+2 -1)
📝 examples/WireMock.Net.ConsoleApplication/MainApp.cs (+13 -7)
📝 src/WireMock.Net.StandAlone/StandAloneApp.cs (+3 -3)
📝 src/WireMock.Net/Admin/Mappings/ResponseModel.cs (+5 -0)
src/WireMock.Net/Http/CertificateUtil.cs (+44 -0)
📝 src/WireMock.Net/Http/HttpClientHelper.cs (+35 -19)
📝 src/WireMock.Net/ResponseBuilders/IProxyResponseBuilder.cs (+2 -2)
📝 src/WireMock.Net/ResponseBuilders/Response.cs (+11 -8)
📝 src/WireMock.Net/Server/FluentMockServer.Admin.cs (+13 -3)
📝 src/WireMock.Net/Settings/ProxyAndRecordSettings.cs (+2 -2)
📝 test/WireMock.Net.Tests/FluentMockServerTests.cs (+18 -1)

📄 Description

Add support for client certificate password and test with real services that require client certificates. Also update so when proxying, the path and query will be passed to the proxied api.

Please have a look and let me know what you think. I ran a test for it locally, but it's a bit hard to checkin a runnable unit test as it would require a real service (which I have in my environment) plus a valid pfx with password


🔄 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/32 **Author:** [@phillee007](https://github.com/phillee007) **Created:** 6/10/2017 **Status:** ✅ Merged **Merged:** 6/16/2017 **Merged by:** [@StefH](https://github.com/StefH) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (10+) - [`98eed7f`](https://github.com/wiremock/WireMock.Net/commit/98eed7f34dcb6ab12d97a5f49b3845587047dabd) Add support for client certificate password and test with some real services that require client certificates. Also update so when proxying, the path and query will be passed to the proxied api. - [`3c26f44`](https://github.com/wiremock/WireMock.Net/commit/3c26f4466281fbd0d2321f0aef0e7f6412665957) Add correct param - [`f3a11fa`](https://github.com/wiremock/WireMock.Net/commit/f3a11fa62c705225f126c1db93a857447e969d74) Update to read certificate from store instead of file - [`4d9e928`](https://github.com/wiremock/WireMock.Net/commit/4d9e9288c539d9cf845d347bbf589eb7483fc3bb) Merge branch 'master' of https://github.com/StefH/WireMock.Net - [`3358e8e`](https://github.com/wiremock/WireMock.Net/commit/3358e8e0f8132448350a005f9c49b400181d50a0) Add support for client certificate password and test with some real services that require client certificates. Also update so when proxying, the path and query will be passed to the proxied api. - [`70668b2`](https://github.com/wiremock/WireMock.Net/commit/70668b2e3f897c2656f3631cae19e172e46553bb) Add correct param - [`7dc46a1`](https://github.com/wiremock/WireMock.Net/commit/7dc46a151a65fd81e88e273d737094f432507228) Merge branch 'master' of https://github.com/phillee007/WireMock.Net - [`699156a`](https://github.com/wiremock/WireMock.Net/commit/699156a35fda9243bd7ac2ad32d2672a5b59c6b4) Fixup PR issues - [`753522f`](https://github.com/wiremock/WireMock.Net/commit/753522fdae7f08e2ea15a8f572fdb1ce7048c082) Merge branch 'master' of https://github.com/phillee007/WireMock.Net - [`1094af7`](https://github.com/wiremock/WireMock.Net/commit/1094af7228c2ebb8a168826f08368e619d0ad23a) Add support for client certificate password and test with some real services that require client certificates. Also update so when proxying, the path and query will be passed to the proxied api. ### 📊 Changes **11 files changed** (+148 additions, -46 deletions) <details> <summary>View changed files</summary> 📝 `examples/WireMock.Net.Console.Record.NETCoreApp/Program.cs` (+2 -1) 📝 `examples/WireMock.Net.ConsoleApplication/MainApp.cs` (+13 -7) 📝 `src/WireMock.Net.StandAlone/StandAloneApp.cs` (+3 -3) 📝 `src/WireMock.Net/Admin/Mappings/ResponseModel.cs` (+5 -0) ➕ `src/WireMock.Net/Http/CertificateUtil.cs` (+44 -0) 📝 `src/WireMock.Net/Http/HttpClientHelper.cs` (+35 -19) 📝 `src/WireMock.Net/ResponseBuilders/IProxyResponseBuilder.cs` (+2 -2) 📝 `src/WireMock.Net/ResponseBuilders/Response.cs` (+11 -8) 📝 `src/WireMock.Net/Server/FluentMockServer.Admin.cs` (+13 -3) 📝 `src/WireMock.Net/Settings/ProxyAndRecordSettings.cs` (+2 -2) 📝 `test/WireMock.Net.Tests/FluentMockServerTests.cs` (+18 -1) </details> ### 📄 Description Add support for client certificate password and test with real services that require client certificates. Also update so when proxying, the path and query will be passed to the proxied api. Please have a look and let me know what you think. I ran a test for it locally, but it's a bit hard to checkin a runnable unit test as it would require a real service (which I have in my environment) plus a valid pfx with password --- <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:22 +01:00
adam closed this issue 2025-12-29 08:33:22 +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#739