Skip to content

Commit e72128e

Browse files
committed
Refactored code
1 parent 088c3bd commit e72128e

File tree

3 files changed

+23
-21
lines changed

3 files changed

+23
-21
lines changed

src/main/java/in/erail/amazon/lambda/AWSLambda.java

+5-5
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
import com.google.common.base.Strings;
77
import com.google.common.io.ByteStreams;
88
import in.erail.glue.Glue;
9+
import in.erail.model.Event;
910
import in.erail.service.RESTService;
1011
import io.reactivex.Single;
1112
import io.reactivex.schedulers.Schedulers;
@@ -16,7 +17,6 @@
1617
import java.io.OutputStream;
1718
import java.io.OutputStreamWriter;
1819
import in.erail.model.RequestEvent;
19-
import in.erail.model.ResponseEvent;
2020
import java.util.ArrayList;
2121
import java.util.HashSet;
2222
import java.util.List;
@@ -69,11 +69,11 @@ public Single<String> handleMessage(JsonObject pRequest) {
6969
.just(pRequest)
7070
.subscribeOn(Schedulers.computation())
7171
.map(this::convertBodyToBase64)
72-
.map(reqJson -> reqJson.mapTo(RequestEvent.class))
72+
.map(reqJson -> reqJson.mapTo(getService().getRequestEventClass()))
7373
.doOnSuccess(this::populateSystemProperties)
74-
.flatMapMaybe(req -> getService().handleEvent(req))
75-
.toSingle(new ResponseEvent())
76-
.map(resp -> JsonObject.mapFrom(resp))
74+
.flatMapMaybe(req -> getService().handleEvent(getService().createEvent(req)))
75+
.toSingle(new Event())
76+
.map(resp -> JsonObject.mapFrom(resp.getResponse()))
7777
.map(this::sanatizeResponse)
7878
.map(respJson -> respJson.toString());
7979
}

src/main/java/in/erail/amazon/lambda/service/ProxyService.java

+11-8
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package in.erail.amazon.lambda.service;
22

33
import com.google.common.base.Joiner;
4+
import in.erail.model.Event;
45
import in.erail.model.RequestEvent;
56
import in.erail.model.ResponseEvent;
67
import in.erail.server.Server;
@@ -76,12 +77,14 @@ protected String generateURL(RequestEvent pRequest) {
7677
return sb.toString();
7778
}
7879

79-
public MaybeSource<ResponseEvent> process(Maybe<RequestEvent> pRequest) {
80+
public MaybeSource<Event> process(Maybe<Event> pRequest) {
8081
return pRequest.flatMap(this::handle);
8182
}
8283

83-
public Maybe<ResponseEvent> handle(RequestEvent proxyRequest) {
84+
public Maybe<Event> handle(Event pEvent) {
8485

86+
RequestEvent proxyRequest = pEvent.getRequest();
87+
8588
//Build Request
8689
HttpRequest<Buffer> clientRequest = getWebClient().requestAbs(proxyRequest.getHttpMethod(), generateURL(proxyRequest));
8790

@@ -101,24 +104,24 @@ public Maybe<ResponseEvent> handle(RequestEvent proxyRequest) {
101104
return clientRequest
102105
.rxSendBuffer(Buffer.buffer(body))
103106
.map((resp) -> {
104-
ResponseEvent event = new ResponseEvent();
107+
ResponseEvent responseEvent = pEvent.getResponse();
105108

106109
//Add Headers
107110
Optional
108111
.ofNullable(resp.headers())
109112
.orElse(MultiMap.caseInsensitiveMultiMap())
110113
.entries()
111114
.stream()
112-
.forEach((t) -> event.addHeader(t.getKey(), t.getValue()));
115+
.forEach((t) -> responseEvent.addHeader(t.getKey(), t.getValue()));
113116

114-
event.setStatusCode(resp.statusCode());
115-
event.setIsBase64Encoded(true);
117+
responseEvent.setStatusCode(resp.statusCode());
118+
responseEvent.setIsBase64Encoded(true);
116119

117120
Optional
118121
.ofNullable(resp.body())
119-
.ifPresent(b -> event.setBody(b.getBytes()));
122+
.ifPresent(b -> responseEvent.setBody(b.getBytes()));
120123

121-
return event;
124+
return pEvent;
122125
})
123126
.doOnSuccess(e -> getLog().debug(() -> e.toString()))
124127
.toMaybe();

src/main/java/in/erail/service/HelloService.java

+7-8
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
11
package in.erail.service;
22

33
import com.google.common.net.MediaType;
4-
import in.erail.model.RequestEvent;
5-
import in.erail.model.ResponseEvent;
4+
import in.erail.model.Event;
65
import io.reactivex.Maybe;
76
import io.reactivex.MaybeSource;
87
import io.vertx.core.json.JsonArray;
@@ -19,14 +18,14 @@ public HelloService() {
1918
mHelloData.add("S5");
2019
}
2120

22-
2321
@Override
24-
public MaybeSource<ResponseEvent> process(Maybe<RequestEvent> pRequest) {
25-
return Maybe.just(new ResponseEvent()
26-
.setBody(getHelloData().toString().getBytes())
27-
.setMediaType(MediaType.JSON_UTF_8));
22+
public MaybeSource<Event> process(Maybe<Event> pEvent) {
23+
return pEvent.doOnSuccess(e -> e
24+
.getResponse()
25+
.setBody(getHelloData().toString().getBytes())
26+
.setMediaType(MediaType.JSON_UTF_8));
2827
}
29-
28+
3029
public JsonArray getHelloData() {
3130
return mHelloData;
3231
}

0 commit comments

Comments
 (0)