Skip to content

Commit 44fd98d

Browse files
emilyboweLWogan
authored andcommitted
change error to warn (#6294)
This issue appeared while running kafka connection tests which kill the kafka broker. When the connection to the kafka broker was lost, ERROR level logs related to CryptoOpsClientImpl appeared several times. This is because there was a failure in executing net.corda.data.crypto.wire.ops.rpc.queries.SupportedSchemesRpcQuery. It has been verified how this is handled in the rest worker: KeyRestResource calls listSchemes and fails at cryptoOpsClient.getSupportedSchemes. The exception handling for listSchemes will return an InternalServerException. This extends HttpApiException and the response code will be 500. As we return an error to the rest client and the system can recover, the log level for the CryptoOpsClient failing the operation should be WARN level instead.
1 parent 03ef673 commit 44fd98d

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

components/crypto/crypto-client-impl/src/main/kotlin/net/corda/crypto/client/impl/CryptoOpsClientImpl.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -445,7 +445,7 @@ class CryptoOpsClientImpl(
445445
} catch (e: CordaRPCAPIResponderException) {
446446
throw e.toClientException()
447447
} catch (e: Throwable) {
448-
logger.error("Failed executing ${request::class.java.name} for tenant ${context.tenantId}", e)
448+
logger.warn("Failed executing ${request::class.java.name} for tenant ${context.tenantId}", e)
449449
throw e
450450
}
451451
}

0 commit comments

Comments
 (0)