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

large-file-upload: initial addition of azure markdown doc #34

Merged
merged 1 commit into from
Aug 5, 2024

Conversation

youssifsaeed1
Copy link
Contributor

The added document contains the detailed steps needed to set up MS Azure and send files from the Notecard through Notehub to Azure.

The changes can be properly previewed here:-

https://github.com/blues/note-samples/blob/bf80a63478fff9f0977ab4a0c142f94d3744f4f0/python-large-file-upload/doc/azure.md

The added document contains the detailed steps needed to set up MS Azure
and send files from the Notecard through Notehub to Azure.
@youssifsaeed1 youssifsaeed1 added the documentation Improvements or additions to documentation label Aug 1, 2024
- Add the name you want your file to be saved as in Azure. For example, if you want to save your file as 'datafile', modify the URL by adding a forward slash "/" after the storage container name, followed by the desired file name. For example, before modifying, the URL looks like this:-

`
https://ysazurestorage.blob.core.windows.net/bluesfilescontainer?sp=racw&st=2024-08-01T11:15:56Z&se=2024-08-01T19:15:56Z&spr=https&sv=2022-11-02&sr=c&sig=Kzo3CL%2BkVm74PTdHM5OoxbjsdHyuc1c1KwDnfkcBjJg%3D
Copy link
Collaborator

Choose a reason for hiding this comment

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

Does this contain any "secrets" or unique IDs?

If so, you may want to replace some of the characters of the unique ID with xxxxxxxxxxxxxxxxxxxx or something similar to indicate it's a unique ID, but don't use this specific one.

It's not clear to me, so feel free to ignore this comment if I'm off base.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You're right it does, but this specific SAS key was set to expire 24h after it was generated so it is no longer usable.

curl -i -X PUT \
-H "x-ms-blob-type: BlockBlob"
--upload-file myfile.txt \
"https://iotcsayau4ueernijnm.blob.core.windows.net/test-container-20240719/myfile.txt?sp=r&st=2024-07-19T21:20:38Z&se=2024-07-20T05:20:38Z&spr=https&sv=2022-11-02&sr=c&sig=muUfzJZG1K635lTDft0yFyQm4ryQxFU9LsXH4XzrU28%3D"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Contain Unique ID?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Same as above

@youssifsaeed1 youssifsaeed1 merged commit b19c75c into main Aug 5, 2024
3 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants