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

Add Azure Blob Storage as a PolicyBundleServerType #531

Closed
wants to merge 5 commits into from

Conversation

devine12
Copy link

@devine12 devine12 commented Jan 4, 2024

Fixes Issue

Fix issue #529

Changes proposed

Ability to connect to Azure Blob Storage as the PolicyBundleServerType.

Ability to generate SAS tokens dynamically via ManagedIdentity.

Add URL encoding to the api_policy_source, fetch_policy_bundle_from_api_source method due to special characters in the SAS tokens required to connect to Azure Blob Storage.

Check List (Check all the applicable boxes)

  • I sign off on contributing this submission to open-source
  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

Copy link

netlify bot commented Jan 4, 2024

Deploy Preview for opal-docs ready!

Name Link
🔨 Latest commit 246e844
🔍 Latest deploy log https://app.netlify.com/sites/opal-docs/deploys/65971fd55930bb0008bf1369
😎 Deploy Preview https://deploy-preview-531--opal-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@devine12 devine12 closed this Jan 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant