zydronium/google-nomulus
1
0
Fork 0
mirror of https://github.com/google/nomulus.git synced 2025-05-13 16:07:15 +02:00
Code Issues Projects Releases Packages Wiki Activity

Log all exceptions thrown at the end of the pipeline

Browse source 
The RelayHandler is installed at the end of a channel pipeline (both frontend and backend). If it does not log the exception, it will be regarded and unhandled exception, which shows up in logs, but does not log the corresponding channel.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=208984756
This commit is contained in:
jianglai 2018-08-16 08:05:30 -07:00
parent 8a1c99e22b
commit 782643ce33
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
java/google/registry/proxy/handler/RelayHandler.java
View file

@ -78,10 +78,11 @@ public class RelayHandler<I> extends SimpleChannelInboundHandler<I> {
if (cause instanceof OverQuotaException) { if (cause instanceof OverQuotaException) {
logger.atWarning().withCause(cause).log( logger.atWarning().withCause(cause).log(
"Channel %s closed due to quota exceeded.", ctx.channel()); "Channel %s closed due to quota exceeded.", ctx.channel());
ChannelFuture unusedFuture = ctx.close();
} else { } else {
ctx.fireExceptionCaught(cause); logger.atWarning().withCause(cause).log(
"Channel %s closed due to unexpected exception.", ctx.channel());
} }
ChannelFuture unusedFuture = ctx.close();
} }
public static void writeToRelayChannel( public static void writeToRelayChannel(