-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Add Azure Blob Storage tasks #1538
Conversation
Here I am, brain the size of a planet and they ask me to welcome you to Prefect. So, welcome to the community @frsann! 🎉 🎉 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks really great! Would you also mind adding this Task and yourself to CHANGELOG.md
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome! This is a great addition! Thanks @frsann
Hi, One key improvement would be to allow users to use a SAS token rather than the ACCOUNT_KEY
Using SAS tokens is considered more secure than having to share you ACCOUNT_NAME and ACCOUNT_KEY with potentially untrusted services. See more details here: https://docs.microsoft.com/en-us/azure/storage/common/storage-sas-overview See also the emulator that may be useful for testing without the cost implications of using real credentials. Thanks |
@dazzag24 Great suggestion! Feel free to open a feature request! |
Thanks for contributing to Prefect!
Please describe your work and make sure your PR:
CHANGELOG.md
(if appropriate)docs/outline.toml
for API reference docs (if appropriate)Note that your PR will not be reviewed unless all three boxes are checked.
What does this PR change?
Adds Azure Blob Storage tasks.
Why is this PR important?
Azure Blob Storage users can upload and download stuff.