Skip to content

Commit 5d6bc5f

Browse files
committed
fix: Actionable error utility determines DEBUG log level internally
1 parent 8294d78 commit 5d6bc5f

2 files changed

Lines changed: 3 additions & 7 deletions

File tree

gax-java/gax/src/main/java/com/google/api/gax/logging/LoggingUtils.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -152,17 +152,13 @@ public static <RespT> void logRequest(
152152
* @param logContext A map containing the structured logging context (e.g., RPC service, method,
153153
* error details).
154154
* @param loggerProvider The provider used to obtain the logger.
155-
* @param level The slf4j level to log the actionable error at.
156155
* @param message The human-readable error message.
157156
*/
158157
public static void logActionableError(
159-
Map<String, Object> logContext,
160-
LoggerProvider loggerProvider,
161-
org.slf4j.event.Level level,
162-
String message) {
158+
Map<String, Object> logContext, LoggerProvider loggerProvider, String message) {
163159
if (loggingEnabled) {
164160
org.slf4j.Logger logger = loggerProvider.getLogger();
165-
Slf4jUtils.log(logger, level, logContext, message);
161+
Slf4jUtils.log(logger, org.slf4j.event.Level.DEBUG, logContext, message);
166162
}
167163
}
168164

gax-java/gax/src/test/java/com/google/api/gax/logging/LoggingUtilsTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ void testLogActionableError_success() {
115115
when(eventBuilder.addKeyValue(anyString(), any())).thenReturn(eventBuilder);
116116

117117
Map<String, Object> context = Collections.singletonMap("key", "value");
118-
LoggingUtils.logActionableError(context, loggerProvider, org.slf4j.event.Level.DEBUG, "message");
118+
LoggingUtils.logActionableError(context, loggerProvider, "message");
119119

120120
verify(loggerProvider).getLogger();
121121
}

0 commit comments

Comments
 (0)