Skip to content

Commit 866e6b8

Browse files
committed
refactor: change log level from warning to fine for deserialization errors in event and request buses
1 parent 8c72dd6 commit 866e6b8

File tree

2 files changed

+6
-12
lines changed

2 files changed

+6
-12
lines changed

src/main/kotlin/dev/slne/surf/redis/event/RedisEventBus.kt

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ class RedisEventBus internal constructor(private val api: RedisApi) {
109109
val envelope = try {
110110
api.json.decodeFromString<EventEnvelope>(message)
111111
} catch (e: SerializationException) {
112-
log.atWarning()
112+
log.atFine()
113113
.withCause(e)
114114
.log("Unable to deserialize event envelope: ${e.message}")
115115
return
@@ -118,9 +118,7 @@ class RedisEventBus internal constructor(private val api: RedisApi) {
118118
val eventClass = eventTypeRegistry[envelope.eventClass]
119119

120120
if (eventClass == null) {
121-
log.atSevere()
122-
.per(envelope.eventClass, LogPerBucketingStrategy.byHashCode(128))
123-
.atMostEvery(1, TimeUnit.MINUTES)
121+
log.atFine()
124122
.log("No registered event class for name: ${envelope.eventClass} - ignoring event.")
125123
return
126124
}

src/main/kotlin/dev/slne/surf/redis/request/RequestResponseBus.kt

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ class RequestResponseBus internal constructor(
108108
val envelope = try {
109109
api.json.decodeFromString<RequestEnvelope>(message)
110110
} catch (e: SerializationException) {
111-
log.atWarning()
111+
log.atFine()
112112
.withCause(e)
113113
.log("Unable to deserialize request envelope: ${e.message}")
114114
return
@@ -117,9 +117,7 @@ class RequestResponseBus internal constructor(
117117
val requestClass = requestTypeRegistry[envelope.requestClass]
118118

119119
if (requestClass == null) {
120-
log.atSevere()
121-
.per(envelope.requestClass, LogPerBucketingStrategy.byHashCode(128))
122-
.atMostEvery(1, TimeUnit.MINUTES)
120+
log.atFine()
123121
.log("No registered request class for name: ${envelope.requestClass} - ignoring request.")
124122
return
125123
}
@@ -153,7 +151,7 @@ class RequestResponseBus internal constructor(
153151
val envelope = try {
154152
api.json.decodeFromString<ResponseEnvelope>(message)
155153
} catch (e: SerializationException) {
156-
log.atWarning()
154+
log.atFine()
157155
.withCause(e)
158156
.log("Unable to deserialize response envelope: ${e.message}")
159157
return
@@ -162,9 +160,7 @@ class RequestResponseBus internal constructor(
162160
val responseClass = responseTypeRegistry[envelope.responseClass]
163161

164162
if (responseClass == null) {
165-
log.atWarning()
166-
.per(envelope.responseClass, LogPerBucketingStrategy.byHashCode(128))
167-
.atMostEvery(1, TimeUnit.MINUTES)
163+
log.atFine()
168164
.log("No registered response class for name: ${envelope.responseClass} - ignoring response.")
169165
return
170166
}

0 commit comments

Comments
 (0)