Azure / Azure/azure-storage-python

Issue when generating a url using sas token. Method file_service.make_file_url()

Open
#609 9 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
343
Forks
243
PR merge metrics
No merged PRs in 30d

Description

### Which service(blob, file, queue) does this issue concern?
File share

### Which version of the SDK was used? Please provide the output of `pip freeze`.
azure-storage-file==2.0.1
azure-storage-common==2.0.0
![sdk-version](https://user-images.githubusercontent.com/32823639/59706469-ea36ce80-91d6-11e9-9dde-670ecbc4e589.PNG)

### What problem was encountered?
https://github.com/Azure/azure-storage-python/blob/b8c5c9a1bc73065fd015c0fec75617cef263c6ee/azure-storage-file/azure/storage/file/fileservice.py#L233

When genrating a url using the method `make_file_url` passing a sas token, the generated url has two query strings ('?') instead of one.
![generated_url](https://user-images.githubusercontent.com/32823639/59705929-c9ba4480-91d5-11e9-8d38-c9809189db2f.PNG)

The SaS Token generated on Azure portal starts with a query string.
![SaSToken-Azure](https://user-images.githubusercontent.com/32823639/59706075-1140d080-91d6-11e9-8f4e-9a45607b1d28.PNG)

### Have you found a mitigation/solution?
Yes. Remove the query string ('?') at line 233.

Note: for table service, please post the issue here instead: https://github.com/Azure/azure-cosmosdb-python.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.