Skip to content

Commit

Permalink
Remove debug logs for ArmeriaServerHttpResponse (#5142)
Browse files Browse the repository at this point in the history
Motivation:

Removed debug logs

Modifications:

- Removed debug logs

Result:

- Unnecessary debug logs don't confuse users

<!--
Visit this URL to learn more about how to write a pull request
description:

https://armeria.dev/community/developer-guide#how-to-write-pull-request-description
-->
  • Loading branch information
jrhee17 authored Aug 25, 2023
1 parent ee42213 commit 8504e95
Showing 1 changed file with 0 additions and 16 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -88,22 +88,6 @@ private Mono<Void> write(Flux<? extends DataBuffer> publisher) {
publisher.map(factoryWrapper::toHttpData)
.contextWrite(contextView)
.doOnDiscard(PooledDataBuffer.class, DataBufferUtils::release)
.doOnCancel(() -> {
logger.debug("{} Response stream cancelled", ctx,
new RuntimeException());
})
.doOnError(cause -> {
logger.debug("{} Response stream aborted. cause: {}", ctx,
cause, new RuntimeException());
})
.doOnComplete(() -> {
logger.debug("{} Response stream completed", ctx,
new RuntimeException());
})
.doFinally(signalType -> {
logger.debug("{} Response stream has been finished", ctx,
new RuntimeException());
})
);
future.complete(response);
return Mono.fromFuture(response.whenComplete())
Expand Down

0 comments on commit 8504e95

Please sign in to comment.