Skip to content

Commit f73af55

Browse files
author
innokenty
committed
create http client once per user request
1 parent 1e3517d commit f73af55

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

proxy/src/main/java/ru/qatools/gridrouter/RouteServlet.java

+3-1
Original file line numberDiff line numberDiff line change
@@ -103,14 +103,15 @@ protected void doPost(HttpServletRequest request, HttpServletResponse response)
103103
List<Region> unvisitedRegions = new ArrayList<>(allRegions);
104104

105105
int attempt = 0;
106+
CloseableHttpClient client = newHttpClient();
106107
while (!allRegions.isEmpty()) {
107108
attempt++;
108109

109110
Region currentRegion = hostSelectionStrategy.selectRegion(allRegions, unvisitedRegions);
110111
Host host = hostSelectionStrategy.selectHost(currentRegion.getHosts());
111112

112113
String route = host.getRoute();
113-
try (CloseableHttpClient client = newHttpClient()) {
114+
try {
114115
LOGGER.info("[SESSION_ATTEMPTED] [{}] [{}] [{}] [{}] [{}]", user, remoteHost, browser, route, attempt);
115116

116117
String target = route + request.getRequestURI();
@@ -146,6 +147,7 @@ protected void doPost(HttpServletRequest request, HttpServletResponse response)
146147
unvisitedRegions = new ArrayList<>(allRegions);
147148
}
148149
}
150+
client.close();
149151

150152
LOGGER.error("[SESSION_NOT_CREATED] [{}] [{}] [{}]", user, remoteHost, browser);
151153
replyWithError("Cannot create session on any available node", response);

0 commit comments

Comments
 (0)