-
Notifications
You must be signed in to change notification settings - Fork 284
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 storage account upload support #5
base: master
Are you sure you want to change the base?
Conversation
Adding feature for Azure storage uploading.
Just my random 2 cents. This could be done without bringing in Azure dependencies. With a shared access signature URL files can be saved in storage accounts with a plain old PUT request. Additionally, it means you can created a SAS token with only access to write new blobs without needing the storage account key with full access. It could also done in a generic way that supports POST or PUT to any URL. |
@coderpatros I don't personally think the fact the change utilises Azure dependencies matters? The existing AWS and GCP exports both have dependencies on their retrospective CLI tools. |
@jusso-dev Just putting it out there that a bunch of extra bundled code, in the form of dependencies, could be dropped with a few lines of standard library code. |
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 comment has been deleted]
Cloudflare master
This change adds support for uploading scan reports to Azure storage accounts.
The primary downside to this change is that the Azure python packages have a dependency on cffi, which performs a compile when installed in alpine. Thus, it requires gcc and some other support libraries to complete installation.
build-base
is removed again after the pip installations complete.