QueryString should not be in the wrap authentication

[andrerod]

This will cause cache to be refreshing more than needed.

#86

[andrerod]

Can be done later.

[paed01]

Regarding module Wrap: lib/services/serviceBus/internal/wrap.js

Shouldn’t the signRequest function be something like this to prevent unnecessary calls to acs:

Wrap.prototype.signRequest = function (webResource, callback) {
var parsedUrl = url.parse(webResource.requestUrl);
parsedUrl.protocol = 'http:';
delete parsedUrl.pathname;
delete parsedUrl.host;
delete parsedUrl.port;
delete parsedUrl.search;
var requestUrl = url.format(parsedUrl);

The committed function tries to delete parsedUrl.path property. That property is not documented in the url-module. The search property deletes the query string.

[yugangw-msft]

closing as part of xplat backlog scrub