Skip to content
This repository was archived by the owner on Jul 19, 2024. It is now read-only.

Commit dae672f

Browse files
committed
Resolved GitHub Issue #431
1 parent e463b46 commit dae672f

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

src/main/java/com/microsoft/azure/storage/blob/LoggingFactory.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -159,8 +159,8 @@ public Single<HttpResponse> sendAsync(final HttpRequest request) {
159159
}
160160

161161
if (this.shouldLog(HttpPipelineLogLevel.INFO)) {
162-
String logMessage = String.format("'%s'==> OUTGOING REQUEST (Try number='%d')%n", request.url(),
163-
this.tryCount);
162+
String logMessage = String.format("'%s'==> OUTGOING REQUEST (Try number='%d')%n",
163+
sanitizeURL(request.url()), this.tryCount);
164164
this.log(HttpPipelineLogLevel.INFO, logMessage);
165165
}
166166

src/test/java/com/microsoft/azure/storage/blob/LoggingTest.groovy

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -405,7 +405,7 @@ class LoggingTest extends APISpec {
405405
then:
406406
1 * logger.log(HttpPipelineLogLevel.INFO, _, _) >>
407407
{ HttpPipelineLogLevel level, String message, Object[] params ->
408-
if (!message.contains("OUTGOING REQUEST")) {
408+
if (!message.contains("OUTGOING REQUEST") || message.contains("urlSignature")) {
409409
throw new IllegalArgumentException(message)
410410
}
411411
}

0 commit comments

Comments
 (0)