Skip to content

Commit e677cee

Browse files
committed
Minor refactoring
1 parent df6cd04 commit e677cee

2 files changed

Lines changed: 8 additions & 3 deletions

File tree

src/main/java/in/erail/route/OpenAPI3RouteBuilder.java

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
import com.codahale.metrics.MetricRegistry;
44
import com.codahale.metrics.Timer;
5+
import com.google.common.base.Strings;
56
import com.google.common.net.HttpHeaders;
67
import com.google.common.net.MediaType;
78
import java.io.File;
@@ -102,7 +103,10 @@ public JsonObject serialiseRoutingContext(RoutingContext pContext) {
102103
if (pContext.request().method() == HttpMethod.POST) {
103104
boolean bodyAsJson = pContext.<Boolean>get(FrameworkConstants.RoutingContext.Attribute.BODY_AS_JSON);
104105
if (bodyAsJson) {
105-
result.put(FrameworkConstants.RoutingContext.Json.BODY, pContext.getBodyAsJson());
106+
String contentType = pContext.request().headers().get(HttpHeaders.CONTENT_TYPE);
107+
if (!Strings.isNullOrEmpty(contentType) && contentType.equals(MediaType.JSON_UTF_8.toString())) {
108+
result.put(FrameworkConstants.RoutingContext.Json.BODY, pContext.getBodyAsJson());
109+
}
106110
} else {
107111
result.put(FrameworkConstants.RoutingContext.Json.BODY, pContext.getBody().getDelegate().getBytes());
108112
}
@@ -158,7 +162,7 @@ public Map<String, Object> convertMultiMapIntoMap(MultiMap pMultiMap) {
158162
.getDelegate()
159163
.entries()
160164
.stream()
161-
.collect(Collectors.toMap((t) -> t.getKey(), (t) -> t.getValue()));
165+
.collect(Collectors.toMap((t) -> t.getKey(), (t) -> t.getValue(), (a, b) -> a));
162166
}
163167

164168
public DeliveryOptions getDeliveryOptions() {

src/test/java/in/erail/service/BroadcastServiceTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package in.erail.service;
22

33
import com.google.common.net.HttpHeaders;
4+
import com.google.common.net.MediaType;
45
import in.erail.server.Server;
56
import in.erail.test.TestConstants;
67
import org.junit.Test;
@@ -45,7 +46,7 @@ public void testProcess(TestContext context) {
4546
.getVertx()
4647
.createHttpClient()
4748
.post(server.getPort(), server.getHost(), "/v1/broadcast/testTopic")
48-
.putHeader("content-type", "application/json")
49+
.putHeader("content-type", MediaType.JSON_UTF_8.toString())
4950
.putHeader(HttpHeaders.ORIGIN, "https://test.com")
5051
.putHeader("content-length", Integer.toString(json.length()))
5152
.putHeader(HttpHeaders.AUTHORIZATION, TestConstants.ACCESS_TOKEN)

0 commit comments

Comments
 (0)