Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Storage][Blob][Bug]Support parsing blob url with '/' in blob name #12619

Merged
merged 5 commits into from
Aug 12, 2020

Conversation

xiafu-msft
Copy link
Contributor

@xiafu-msft xiafu-msft commented Jul 20, 2020

@ghost ghost added the Storage Storage Service (Queues, Blobs, Files) label Jul 20, 2020
@xiafu-msft xiafu-msft added the bug This issue requires a change to an existing behavior in the product in order to be resolved. label Jul 21, 2020
@scbedd
Copy link
Member

scbedd commented Jul 21, 2020

/azp run python - storage - ci

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@@ -320,11 +325,11 @@ def __exit__(self, *args): # pylint: disable=arguments-differ
pass


def format_shared_key_credential(account, credential):
def format_shared_key_credential(account_name, credential):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should rename this to _format_shared_key_credential . It's not exposed so rather make it private

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done!

container_client = ContainerClient(
account_url='http://127.0.0.1:1000/devstoreaccount1',
container_name='newcontainer',
credential='Eby8vdM02xNOcqFlqUwJPLlmEtlCDXJ1OUzFT50uSRZ6IFsuFq2UVErCz4I6tq/K1SZFPTOtr/KBHBeksoGMGw==')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hoping this is not real :P

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's not

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I bet we'll get email to remove it soon :P

@xiafu-msft xiafu-msft merged commit a1039c8 into Azure:master Aug 12, 2020
iscai-msft added a commit to iscai-msft/azure-sdk-for-python that referenced this pull request Aug 17, 2020
…into get_certs_2016

* 'master' of https://github.com/Azure/azure-sdk-for-python: (26 commits)
  Type inference (Azure#12980)
  Add azure-ai-anomalydetector to error code 5 ignored list (Azure#13131)
  Handle missing model properties in Key Vault 2016-10-01 (Azure#13129)
  fix aad test so it runs locally (Azure#13127)
  Suppress errors in anomoly detector for now. (Azure#13141)
  Anomaly Detector V1, track2 generator (Azure#12931)
  [formrecognizer] Add `kind` to FormElement (Azure#13079)
  Regenerate code from connie's swagger change (Azure#12979)
  Update async (Azure#12978)
  Move get_access_conditions to _serialize.py (Azure#13105)
  [Storage][DataLake]Update Min Dependency (Azure#13108)
  Restore user authentication API from 1.4.0b7 (Azure#13070)
  [Storage][FileShare]ChangeLog update (Azure#13103)
  remove locale (Azure#13102)
  [Storage][Blob]ChangeLog Update (Azure#13081)
  Fixed etag bug (Azure#13078)
  [Storage][Blob]Fix live test and if tags bug (Azure#13054)
  [Storage][Blob][Batch]Support batch delete empty blob list (Azure#13029)
  [Storage][Blob][Bug]Support parsing blob url with '/' in blob name (Azure#12619)
  [formrecognizer] add api version enum (Azure#12888)
  ...
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-python that referenced this pull request Feb 8, 2021
[Hub Generated] Review request for Microsoft.KeyVault to add version preview/2020-04-01-preview (Azure#12619)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue requires a change to an existing behavior in the product in order to be resolved. Storage Storage Service (Queues, Blobs, Files)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants