Skip to content

[Tables] Add multitenant challenge auth policy support #24278

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.

Sign up for GitHub

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

Merged
mccoyp merged 10 commits into from
May 6, 2022
Merged

[Tables] Add multitenant challenge auth policy support #24278

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
f49b7a2
Add challenge auth policies
mccoyp May 3, 2022
7de91d8
Add tests
mccoyp May 3, 2022
63c8d86
Add recorded tests
mccoyp May 3, 2022
d1fe634
Clean up environment variables
mccoyp May 3, 2022
9acbba8
Update CHANGELOG.md
mccoyp May 5, 2022
d5add7f
Set tentative release date to avoid error
mccoyp May 5, 2022
68c3eac
Try using "-" instead of "*"?
mccoyp May 5, 2022
efc8c7d
Update _version.py
mccoyp May 5, 2022
68ba174
Update _version.py
mccoyp May 5, 2022
0e31563
Update to 12.4.0
mccoyp May 6, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions sdk/tables/azure-data-tables/CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,10 @@
# Release History

## 12.3.1 (2022-05-10)

### Features Added
* Support for multitenant authentication ([#24278](https://github.com/Azure/azure-sdk-for-python/pull/24278))

## 12.3.0 (2022-03-10)

### Bugs Fixed
Expand Down
2 changes: 1 addition & 1 deletion sdk/tables/azure-data-tables/tests/_shared/asynctestcase.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ class AsyncFakeTokenCredential(object):
def __init__(self):
self.token = AccessToken("YOU SHALL NOT PASS", 0)

async def get_token(self, *args):
async def get_token(self, *args, **kwargs):
return self.token


Expand Down
2 changes: 1 addition & 1 deletion sdk/tables/azure-data-tables/tests/_shared/testcase.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ class FakeTokenCredential(object):
def __init__(self):
self.token = AccessToken("YOU SHALL NOT PASS", 0)

def get_token(self, *args):
def get_token(self, *args, **kwargs):
return self.token


Expand Down
12 changes: 11 additions & 1 deletion sdk/tables/azure-data-tables/tests/conftest.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,10 @@
# IN THE SOFTWARE.
#
# --------------------------------------------------------------------------
import os

import pytest
from devtools_testutils import add_general_regex_sanitizer, test_proxy
from devtools_testutils import add_general_regex_sanitizer, add_body_key_sanitizer, test_proxy

# fixture needs to be visible from conftest

Expand All @@ -42,3 +44,11 @@ def add_sanitizers(test_proxy):
regex="batch[a-z]*_([0-9a-f]{8}\\b-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-\\b[0-9a-f]{12}\\b)",
group_for_replace="1",
)
# sanitizes tenant ID
tenant_id = os.environ.get("TABLES_TENANT_ID", "00000000-0000-0000-0000-000000000000")
add_general_regex_sanitizer(value="00000000-0000-0000-0000-000000000000", regex=tenant_id)
# sanitizes tenant ID used in test_challenge_auth(_async).py tests
challenge_tenant_id = os.environ.get("CHALLENGE_TABLES_TENANT_ID", "00000000-0000-0000-0000-000000000000")
add_general_regex_sanitizer(value="00000000-0000-0000-0000-000000000000", regex=challenge_tenant_id)
# sanitizes access tokens in response bodies
add_body_key_sanitizer(json_path="$..access_token", value="access_token")
Loading
⚡