Skip to content

Add correlation id, public api id, and current request header #103

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
rayluo merged 16 commits into from
Dec 17, 2019
Merged

Add correlation id, public api id, and current request header #103

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
49fcaea
Add correlation id, public api id, and current request header
Oct 1, 2019
0751be0
Make get_new_correlation_id private
Oct 1, 2019
d61a36a
Update calls to _get_new_correlation_id()
Oct 1, 2019
513d989
Merge branch 'dev' into sagonzal/serverTelemetry
Oct 22, 2019
6c33c42
PR feedback.
Oct 23, 2019
bc414d7
Fix authority test
Oct 25, 2019
1922cb1
PR feedback
Dec 16, 2019
15a0ff4
More PR feedback
Dec 16, 2019
0b58742
Remove indent
sangonzal Dec 17, 2019
2595b75
Merge branch 'dev' into sagonzal/serverTelemetry
Dec 17, 2019
20bf314
Merge branch 'sagonzal/serverTelemetry' of https://github.com/AzureAD…
Dec 17, 2019
4448003
Refactor and fix a bug in the initiate_device_flow()
rayluo Dec 17, 2019
ef15039
Refactor acquire_token_silent() to reuse same correlation_id
rayluo Dec 17, 2019
1652d0a
fixup! Refactor and fix a bug in the initiate_device_flow()
rayluo Dec 17, 2019
e333a46
Define consts for all telemetry headers
rayluo Dec 17, 2019
5135446
Replace one last magic string with const
rayluo Dec 17, 2019
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
Prev Previous commit
Next Next commit
Merge branch 'dev' into sagonzal/serverTelemetry
  • Loading branch information
sgonzalezMSFT committed Oct 22, 2019
commit 513d989580e1b6828723b3705f0bfc6a0f1b86d3
11 changes: 2 additions & 9 deletions msal/authority.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -80,15 +80,8 @@ def __init__(self, authority_url, validate_authority=True,
_, _, self.tenant = canonicalize(self.token_endpoint) # Usually a GUID
self.is_adfs = self.tenant.lower() == 'adfs'

def user_realm_discovery(self, username, headers):
resp = requests.get(
"https://{netloc}/common/userrealm/{username}?api-version=1.0".format(
netloc=self.instance, username=username),
headers=headers.update({'Accept': 'application/json'}),
verify=self.verify, proxies=self.proxies, timeout=self.timeout)
resp.raise_for_status()
return resp.json()
# It will typically contain "ver", "account_type",
def user_realm_discovery(self, username, response=None):
# It will typically return a dict containing "ver", "account_type",
# "federation_protocol", "cloud_audience_urn",
# "federation_metadata_url", "federation_active_auth_url", etc.
if self.instance not in self.__class__._domains_without_user_realm_discovery:
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.