Skip to content

Added support for Service Principal authentication #13

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

roberto-mardeni
Copy link

Purpose

Added support for Service Principal authentication via config.py settings

Does this introduce a breaking change?

[ ] Yes
[ x] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[x ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code
Follow the directions at https://docs.microsoft.com/en-us/azure/batch/quick-run-python on how to test with the Key based authentication
Switch the content of _AUTH_MODE to 'ServicePrincipal' in the src/config.py and provide the values of _AD_* settings (with a valid Service Principal credentials) and run the same script.

What to Check

Verify that the following are valid
Works as usual for the Key based authentication
Works with Service Principal authentication

@hatsem78
Copy link

hatsem78 commented Aug 7, 2020

Tiera erroro por todos lados no existe Traceback (most recent call last):
File "/home/havas/trabajos/batch-python-quickstart/src/python_quickstart_client.py", line 307, in
blob_client = azureblob.BlockBlobService(
AttributeError: module 'azure.storage.blob' has no attribute 'BlockBlobService' actulicen con la nueva versión quedo todo obsolteto

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.

2 participants