bug: always set body to "" when null in MockResponse as clients may fail if content-length etc not set

tommytroen · tommytroen · commit b78660a69d5c · 2021-01-23T19:45:51.000+01:00
diff --git a/src/main/kotlin/no/nav/security/mock/oauth2/http/OAuth2HttpServer.kt b/src/main/kotlin/no/nav/security/mock/oauth2/http/OAuth2HttpServer.kt
@@ -89,7 +89,7 @@ class MockWebServerWrapper : OAuth2HttpServer {
                 .setHeaders(this.headers)
                 .setResponseCode(this.status)
                 .let {
-                    if (this.body != null) it.setBody(this.body) else it
+                    if (this.body != null) it.setBody(this.body) else it.setBody("")
                 }
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -89,7 +89,7 @@ class MockWebServerWrapper : OAuth2HttpServer {`
`89`	`89`	`.setHeaders(this.headers)`
`90`	`90`	`.setResponseCode(this.status)`
`91`	`91`	`.let {`
`92`		`- if (this.body != null) it.setBody(this.body) else it`
	`92`	`+ if (this.body != null) it.setBody(this.body) else it.setBody("")`
`93`	`93`	`}`
`94`	`94`	`}`
`95`	`95`	`}`