Skip to content

Commit 2b6f82b

Browse files
code review
1 parent ab908d9 commit 2b6f82b

File tree

2 files changed

+7
-6
lines changed

2 files changed

+7
-6
lines changed

mvn-resolver-transport-http3/src/main/java/com/artipie/aether/transport/http3/HttpTransporter.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -297,23 +297,23 @@ private Pair<InputStream, HttpFields> makeRequest(
297297
}).body(bodyContent).send(listener);
298298
final Response response = listener.get(this.connectTimeout, TimeUnit.MILLISECONDS);
299299
if (response.getStatus() >= 300) {
300-
LOGGER.error(
300+
LOGGER.debug(
301301
"{} request error status {}, method={}, url={}",
302302
version, response.getStatus(), method, url
303303
);
304304
throw new HttpResponseException(Integer.toString(response.getStatus()), response);
305305
}
306-
LOGGER.info(
306+
LOGGER.debug(
307307
"{} request done, method={}, resp status={}, url={}", version, method, response.getStatus(), url
308308
);
309309
return new ImmutablePair<>(listener.getInputStream(), response.getHeaders());
310310
} catch (Exception ex) {
311-
LOGGER.error(
311+
LOGGER.debug(
312312
"{} request error={}: {}, method={}, url={}", version,
313313
ex.getClass(), ex.getMessage(), method, url
314314
);
315315
if (version == HttpVersion.HTTP_3 && ex instanceof TimeoutException) {
316-
LOGGER.info("Repeat via HTTP/1.1 method={}, url={}", method, url);
316+
LOGGER.debug("Repeat via HTTP/1.1 method={}, url={}", method, url);
317317
return this.makeRequest(method, task, bodyContent, this.initOrGetHttpClient());
318318
}
319319
throw new HttpRequestException(ex.getMessage(), request);

mvn-resolver-transport-http3/src/test/java/com/artipie/aether/transport/http3/ArtipieAndLocalPluginHTTP3IT.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,8 @@ public class ArtipieAndLocalPluginHTTP3IT {
4242
private GenericContainer<?> artipie;
4343

4444
private final Consumer<OutputFrame> artipieLog =
45-
new Slf4jLogConsumer(LoggerFactory.getLogger(this.getClass())).withPrefix("ARTIPIE");
45+
new Slf4jLogConsumer(LoggerFactory.getLogger("ArtipieAndLocalPluginHTTP3IT"))
46+
.withPrefix("ARTIPIE");
4647

4748
private Network net;
4849

@@ -74,7 +75,7 @@ void resolvesDependencies() throws IOException, InterruptedException {
7475
this.putClasspathResourceToClient("com/example/maven-http3/maven-settings.xml", "/w/settings.xml");
7576
this.putClasspathResourceToClient("com/example/maven-http3/pom.xml", "/w/pom.xml");
7677
final Container.ExecResult exec = this.mavenClient.execInContainer(
77-
"mvn", "install", "-s", "settings.xml", "-Daether.connector.https.securityMode=insecure"
78+
"mvn", "install", "-X", "-s", "settings.xml", "-Daether.connector.https.securityMode=insecure"
7879
);
7980
String res = String.join("\n", exec.getStdout(), exec.getStderr());
8081
LOGGER.info(res);

0 commit comments

Comments
 (0)