[PR #828] [MERGED] Add setting to skip saving the string-response in the logging when using WithBody(Func...) #1102

New Issue

Closed

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

adam commented 2025-12-29 08:34:44 +01:00

Owner

Copy Link

📋 Pull Request Information

Original PR: https://github.com/wiremock/WireMock.Net/pull/828
Author: @StefH
Created: 10/13/2022
Status: ✅ Merged
Merged: 10/21/2022
Merged by: @StefH

Base: master ← Head: stef-826

---

📝 Commits (7)

• 9171016 Add extra unit-test for Response WithBody (dynamic code)
• 4899edb Merge branch 'master' into stef-826
• 58066e0 DoNotSaveDynamicResponseInLogEntry
• 7b388fa Merge branch 'master' into stef-826
• 361ba65 update SettingsModel
• 29e1b12 fix
• 86b200e .

📊 Changes

18 files changed (+376 additions, -212 deletions)

View changed files

📝 src/WireMock.Net.Abstractions/Admin/Settings/SettingsModel.cs (+86 -76)
📝 src/WireMock.Net.Abstractions/Models/IBodyData.cs (+5 -2)
📝 src/WireMock.Net/Logging/LogEntry.cs (+4 -4)
📝 src/WireMock.Net/Matchers/LinqMatcher.cs (+1 -0)
📝 src/WireMock.Net/Models/BodyData.cs (+29 -27)
📝 src/WireMock.Net/Owin/IWireMockMiddlewareOptions.cs (+2 -0)
📝 src/WireMock.Net/Owin/WireMockMiddleware.cs (+12 -9)
📝 src/WireMock.Net/Owin/WireMockMiddlewareOptions.cs (+3 -0)
📝 src/WireMock.Net/ResponseBuilders/Response.WithBody.cs (+4 -2)
📝 src/WireMock.Net/ResponseBuilders/Response.cs (+3 -3)
📝 src/WireMock.Net/Serialization/LogEntryMapper.cs (+42 -21)
📝 src/WireMock.Net/Server/WireMockServer.Admin.cs (+8 -3)
📝 src/WireMock.Net/Server/WireMockServer.cs (+1 -0)
📝 src/WireMock.Net/Settings/WireMockServerSettings.cs (+6 -0)
📝 src/WireMock.Net/Settings/WireMockServerSettingsParser.cs (+2 -1)
📝 test/WireMock.Net.Tests/ResponseBuilders/ResponseWithBodyTests.cs (+3 -57)
📝 test/WireMock.Net.Tests/ResponseBuilders/ResponseWithCallbackTests.cs (+113 -4)
📝 test/WireMock.Net.Tests/Serialization/LogEntryMapperTests.cs (+52 -3)

📄 Description

#826

---

_{🔄 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/828 **Author:** [@StefH](https://github.com/StefH) **Created:** 10/13/2022 **Status:** ✅ Merged **Merged:** 10/21/2022 **Merged by:** [@StefH](https://github.com/StefH) **Base:** `master` ← **Head:** `stef-826` --- ### 📝 Commits (7) - [`9171016`](https://github.com/wiremock/WireMock.Net/commit/91710167333e4510826977876db3f9991dd94af7) Add extra unit-test for Response WithBody (dynamic code) - [`4899edb`](https://github.com/wiremock/WireMock.Net/commit/4899edbb0c22aeb76044709efdfdf214f6c3c0d1) Merge branch 'master' into stef-826 - [`58066e0`](https://github.com/wiremock/WireMock.Net/commit/58066e01cee3d2d041a89a2ea3f13f9cc5eea7a8) DoNotSaveDynamicResponseInLogEntry - [`7b388fa`](https://github.com/wiremock/WireMock.Net/commit/7b388fa01854de4ebd1ba188c8e7a82bcbf473da) Merge branch 'master' into stef-826 - [`361ba65`](https://github.com/wiremock/WireMock.Net/commit/361ba65436a5dea77bbc4f14b0f0844d562c2765) update SettingsModel - [`29e1b12`](https://github.com/wiremock/WireMock.Net/commit/29e1b12d6696bae46790bcd0ea2336fb4ce8d3ed) fix - [`86b200e`](https://github.com/wiremock/WireMock.Net/commit/86b200e068d5c4836d304c9a662b0354bbb772c8) . ### 📊 Changes **18 files changed** (+376 additions, -212 deletions) <details> <summary>View changed files</summary> 📝 `src/WireMock.Net.Abstractions/Admin/Settings/SettingsModel.cs` (+86 -76) 📝 `src/WireMock.Net.Abstractions/Models/IBodyData.cs` (+5 -2) 📝 `src/WireMock.Net/Logging/LogEntry.cs` (+4 -4) 📝 `src/WireMock.Net/Matchers/LinqMatcher.cs` (+1 -0) 📝 `src/WireMock.Net/Models/BodyData.cs` (+29 -27) 📝 `src/WireMock.Net/Owin/IWireMockMiddlewareOptions.cs` (+2 -0) 📝 `src/WireMock.Net/Owin/WireMockMiddleware.cs` (+12 -9) 📝 `src/WireMock.Net/Owin/WireMockMiddlewareOptions.cs` (+3 -0) 📝 `src/WireMock.Net/ResponseBuilders/Response.WithBody.cs` (+4 -2) 📝 `src/WireMock.Net/ResponseBuilders/Response.cs` (+3 -3) 📝 `src/WireMock.Net/Serialization/LogEntryMapper.cs` (+42 -21) 📝 `src/WireMock.Net/Server/WireMockServer.Admin.cs` (+8 -3) 📝 `src/WireMock.Net/Server/WireMockServer.cs` (+1 -0) 📝 `src/WireMock.Net/Settings/WireMockServerSettings.cs` (+6 -0) 📝 `src/WireMock.Net/Settings/WireMockServerSettingsParser.cs` (+2 -1) 📝 `test/WireMock.Net.Tests/ResponseBuilders/ResponseWithBodyTests.cs` (+3 -57) 📝 `test/WireMock.Net.Tests/ResponseBuilders/ResponseWithCallbackTests.cs` (+113 -4) 📝 `test/WireMock.Net.Tests/Serialization/LogEntryMapperTests.cs` (+52 -3) </details> ### 📄 Description #826 --- _{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

adam added the pull-request label 2025-12-29 08:34:44 +01:00

adam closed this issue 2025-12-29 08:34:44 +01:00

Sign in to join this conversation.

No Branch/Tag Specified

Branches Tags

master

mime-part-matcher

version-2.x

MappingSerializer

1341-mapping-json

stef-aspire-tests-updates

bug/973-TinyMapper

bug/1149-AppendGuidToSavedMappingFile

feature/1150-DockerImageVersion

61

stef-1108

stef-1097

stef-1083-MessageOptions_Type_Conflict

stef-1062-logger

stef-IgnoreOpenApiErrors

stef-928-TypeLoadException-FluentAssertions-net472

nunit

stef-849

stef-847-regex-questionmark

http_verb

WireMockServerContext

webapp

ai

CommandLineArgumentsParser

1.23.0

1.22.0

1.21.0

1.20.0

1.19.0

1.18.0

1.17.0

1.16.0

1.15.0

1.14.0

1.13.0

1.12.0

1.11.2

1.11.0

1.10.1

1.10.0

1.9.1

1.9.0

1.8.18

1.8.17

1.8.16

1.8.15

1.8.14

1.8.13

1.8.12

1.8.11

1.8.10

1.8.9

1.8.8

1.8.7

1.8.6

1.8.5

1.8.3

1.8.2

1.8.1

1.8.0

1.7.4

1.7.3

1.7.2

1.7.1

1.7.0

1.6.12

1.6.11

1.6.10

1.6.9

1.6.8

1.6.7

1.6.6

1.6.5

1.6.4

1.6.3

1.6.2

1.6.1

1.6.0

1.5.62

1.5.61

1.5.60

1.5.59

1.5.58

1.5.57

1.5.56

1.5.55

1.5.54

1.5.53

1.5.52

1.5.51

1.5.50

1.5.49

1.5.48

1.5.47

1.5.46

1.5.45

1.5.44

1.5.43

1.5.42

1.5.41

1.5.40

1.5.39

1.5.38

1.5.37

1.5.36

1.5.35

1.5.34

1.5.33

1.5.32

1.5.31

1.5.30

1.5.29

1.5.28

1.5.27

1.5.26

1.5.25

1.5.24

1.5.23

1.5.22

1.5.21

1.5.20

1.5.19

1.5.18

1.5.17

1.5.16

1.5.15

1.5.14

1.5.13

1.5.12

1.5.11

1.5.10

1.5.9

1.5.8

1.5.7

1.5.6

1.5.5

1.5.4

1.5.3

1.5.2

1.5.1

1.5.0

1.4.43

1.4.42

1.4.41

1.4.40

1.4.39

1.4.38

1.4.37

1.4.36

1.4.35

1.4.34

1.4.33

1.4.32

1.4.31

1.4.30

1.4.29

1.4.28

1.4.27

1.4.26

1.4.25

1.4.24

1.4.23

1.4.22

1.4.21

1.4.20

1.4.19

1.4.18

1.4.17

1.4.16

1.4.15

1.4.14

1.4.13

1.4.12

1.4.11

1.4.10

1.4.9

1.4.8

1.4.7

1.4.6

1.4.5

1.4.4

1.4.3

1.4.2

1.4.1

1.4.0

1.3.10

1.3.9

1.3.8

1.3.6

1.3.5

1.3.4

1.3.3

1.3.2

1.3.1

1.3.0

1.2.18

1.2.17

1.2.16

1.2.15

1.2.14

1.2.13

1.2.12

1.2.11.0

1.2.10

1.2.9.0

1.2.8.0

1.2.7.0

1.2.6.0

1.2.5.0

1.2.4.0

1.2.3.0

1.2.2.0

1.2.1.0

1.2.0.0

1.1.10

1.1.9.0

1.1.8.0

1.1.7.0

1.1.6.0

1.1.5.0

1.1.3.0

1.1.2.0

1.1.1.0

1.1.0.0

1.0.43.0

1.0.42.0

1.0.41.0

1.0.40.0

1.0.39.0

1.0.38.0

1.0.37.0

1.0.36.0

1.0.35.0

1.0.34.0

1.0.33.0

1.0.32.0

1.0.31.0

1.0.29.0

1.0.28.0

1.0.25.0

1.0.24.0

1.0.23.0

1.0.22.0

1.0.21.0

1.0.20.0

1.0.19.0

1.0.18.0

1.0.17.0

1.0.16.0

1.0.15.0

1.0.14.0

1.0.13.0

1.0.12.0

1.0.11.0

1.0.10.0

1.0.9.0

1.0.8.0

1.0.7.0

1.0.6.1

1.0.6

1.0.5

1.0.4.21

1.0.4.20

1.0.4.19

1.0.4.18

1.0.4.17

1.0.4.16

1.0.4.15

1.0.4.14

1.0.4.13

1.0.4.12

1.0.4.11

1.0.4.10

1.0.4.9

1.0.4.8

1.0.4.7

1.0.4.6

1.0.4.5

1.0.4.4

1.0.4.3

1.0.4.2

1.0.4.1

1.0.4.0

1.0.3.20

1.0.3.19

1.0.3.18

1.0.3.17

1.0.3.16

1.0.3.15

1.0.3.14

1.0.3.13

1.0.3.12

1.0.3.11

1.0.3.10

1.0.3.9

1.0.3.8

1.0.3.7

1.0.3.6

1.0.3.5

1.0.3.4

1.0.3.3

1.0.3.2

1.0.3.1

1.0.3.0

1.0.2.13

1.0.2.12

1.0.2.11

1.0.2.10

1.0.2.9

1.0.2.8

1.0.2.7

1.0.2.6

1.0.2.5

1.0.2.4

1.0.2.1

1.0.2.0

1.0.1.5

1.0.1.3

1.0.1.2

1.0.1.1

1.0.0.0

Labels

Clear labels

bug

bug

doc

doc

duplicate

feature

feature

invalid

pull-request

Mirrored from GitHub Pull Request

question

wontfix

No Label pull-request

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

adam

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/WireMock.Net#1102