Merge pull request Azure#439 from seanmcc-msft/431

Resolved GitHub Issue Azure#431
g2vinay · Feb 19, 2019 · 15f77f3 · 15f77f3
2 parents e463b46 + dae672f
commit 15f77f3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/main/java/com/microsoft/azure/storage/blob/LoggingFactory.java b/src/main/java/com/microsoft/azure/storage/blob/LoggingFactory.java
@@ -159,8 +159,8 @@ public Single<HttpResponse> sendAsync(final HttpRequest request) {
             }
 
             if (this.shouldLog(HttpPipelineLogLevel.INFO)) {
-                String logMessage = String.format("'%s'==> OUTGOING REQUEST (Try number='%d')%n", request.url(),
-                        this.tryCount);
+                String logMessage = String.format("'%s'==> OUTGOING REQUEST (Try number='%d')%n",
+                        sanitizeURL(request.url()), this.tryCount);
                 this.log(HttpPipelineLogLevel.INFO, logMessage);
             }
 

diff --git a/src/test/java/com/microsoft/azure/storage/blob/LoggingTest.groovy b/src/test/java/com/microsoft/azure/storage/blob/LoggingTest.groovy
@@ -405,7 +405,7 @@ class LoggingTest extends APISpec {
         then:
         1 * logger.log(HttpPipelineLogLevel.INFO, _, _) >>
                 { HttpPipelineLogLevel level, String message, Object[] params ->
-                    if (!message.contains("OUTGOING REQUEST")) {
+                    if (!message.contains("OUTGOING REQUEST") || message.contains("urlSignature")) {
                         throw new IllegalArgumentException(message)
                     }
                 }