Fix 401 Unauthorized error in CosmosDB operations

[spica314]

fixes #831

[cataggar]

This looks fine to me. I'd like @rylev to review. Would prefer a test added. I wonder if the query part is actually needed for the AuthorizationPolicy.

[cataggar]

This matches what the code was, prior to this change:
https://github.com/Azure/azure-sdk-for-rust/pull/811/files#diff-1d8d32914ddb120499d9acbf8fb856f3757b01bf2d612c4eef44e5ff9590dcbaL57

I think in #811, there was a switch from http::Uri to url::Url and path_and_query existed in the former.

https://docs.rs/http/latest/http/uri/struct.Uri.html#method.path_and_query
https://docs.rs/url/latest/url/struct.Url.html#method.path

[rylev]

Whoops! Good find. I think we should probably change the generate_resource_link to take a &Request and have that function do the magical &request.path_and_query()[1..] dance. That way it's all contained within that function and not the callers responsibility to give the generate_resource_link function a "uri path" in the right format.

I also agree that it would be nice to have a test for this. Maybe we can change the tests at the end of the file to generate the resource_link instead of hard coding it?

[ghost]

All CLA requirements met.

[spica314]

@rylev
Thank you for your review!

I moved the code for striping the leading slash to generate_resource_link.
(Since my English skill is poor, if my comments are incorrect, please tell me...)

And I also rewrite tests for generate_resource_link, but I'm not sure whether I'm following your suggestion.
If not, please tell me more (example or functions to use).

[rylev]

Thank you! 🎉