Skip to content

Commit 80fe91d

Browse files
committed
build user agent only once
1 parent b0468e2 commit 80fe91d

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

rmf/rmf-java-base/src/main/java/io/vrap/rmf/base/client/middlewares/HttpMiddleware.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ public class HttpMiddleware implements Middleware {
1616
private final String apiBaseUrl;
1717
private CompletableFuture<AuthenticationToken> authenticationToken;
1818
private final VrapHttpClient httpClient;
19+
private final String userAgent;
1920

2021
public HttpMiddleware(
2122
final String apiBaseUrl,
@@ -25,6 +26,7 @@ public HttpMiddleware(
2526
this.apiBaseUrl = removeTrailingSlash(apiBaseUrl);
2627
this.tokenSupplier = tokenSupplier;
2728
this.httpClient = httpClient;
29+
this.userAgent = buildUserAgent();
2830
}
2931

3032
@Override
@@ -33,7 +35,7 @@ public CompletableFuture<MiddlewareArg> next(MiddlewareArg arg) {
3335
return arg.getNext().next(arg);
3436
}
3537
ApiHttpRequest request = arg.getRequest();
36-
request.addHeader(USER_AGENT, getUserAgent());
38+
request.addHeader(USER_AGENT, userAgent);
3739
request.setBaseUrl(apiBaseUrl);
3840
return execute(request, null, 0)
3941
.thenApply(response ->
@@ -91,7 +93,7 @@ private String removeTrailingSlash(String url) {
9193
return url;
9294
}
9395

94-
private String getUserAgent() {
96+
private String buildUserAgent() {
9597
String runtimeVersion = SystemUtils.JAVA_RUNTIME_VERSION;
9698
String osName = SystemUtils.OS_NAME;
9799
String osArch = SystemUtils.OS_ARCH;

0 commit comments

Comments
 (0)