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

Manual Read Region Selection - Testing #2037

Draft
wants to merge 30 commits into
base: master
Choose a base branch
from

Conversation

bogercraig
Copy link
Contributor

@bogercraig bogercraig commented Jan 29, 2025

Preview this branch

PR to test read region selection.

Read region selection is supported for NoSQL accounts using AAD auth.

…wn menu. Update local storage value.

Need to now connect with updating read region of primary cosmos client.
…t. Still need to:

1 - Use separate context var to track selected region.  Directly updating database account context throws off token generation by acquireMSALTokenForAccount
2 - Remove href overrides in acquireMSALTokenForAccount.
…que list of read and write endpoints.

Need to continue with clearing out selected endpoint when global is selected again.
Write operations stall when read region is selected even though 403 returned when region rejects operation.
Need to limit feature availablility to nosql, table, gremlin (maybe).
Clients continuously generate after changing RBAC setting.
…onments. Passes new error to calling code.

Might need to add specific error handling for request plugin to the handleError class.
…ly. Could possibly enable on table and gremlin later.
@bogercraig bogercraig force-pushed the users/bogercraig/manual_read_region_selection branch from 37eacd3 to 6786ba5 Compare January 31, 2025 20:49
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