Skip to content

Commit 924456b

Browse files
committed
resolves #2004 Improve logging in background tasks
1 parent de6ac88 commit 924456b

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

bot/connector-mattermost/src/main/kotlin/MattermostClient.kt

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ package ai.tock.bot.connector.mattermost
1919
import ai.tock.bot.connector.mattermost.model.MattermostMessageOut
2020
import ai.tock.shared.addJacksonConverter
2121
import ai.tock.shared.create
22+
import ai.tock.shared.error
2223
import ai.tock.shared.longProperty
2324
import ai.tock.shared.retrofitBuilderWithTimeoutAndLogger
2425
import mu.KotlinLogging
@@ -56,7 +57,11 @@ internal class MattermostClient(
5657
.create()
5758

5859
fun sendMessage(message: MattermostMessageOut) {
59-
val response = mattermostApi.sendMessage(mattermostToken, message).execute()
60-
logger.debug { response }
60+
try {
61+
val response = mattermostApi.sendMessage(mattermostToken, message).execute()
62+
logger.debug { response }
63+
} catch (e: Throwable) {
64+
logger.error(e)
65+
}
6166
}
6267
}

shared/src/main/kotlin/vertx/VertXs.kt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -194,6 +194,7 @@ internal fun vertxExecutor(): Executor {
194194
val v = blocking.call()
195195
promise.complete(v)
196196
} catch (e: Throwable) {
197+
logger.error(e)
197198
promise.fail(e)
198199
} finally {
199200
promise.tryFail("call not completed")

0 commit comments

Comments
 (0)