Merge pull request #439 from seanmcc-msft/431

Resolved GitHub Issue #431

Author: rickle-msft

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);
             }
 

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)
                     }
                 }