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

[Communication] Fix async identity client decode - url bug #15894

Merged
merged 4 commits into from
Dec 22, 2020

Conversation

turalf
Copy link
Contributor

@turalf turalf commented Dec 21, 2020

No description provided.

@turalf
Copy link
Contributor Author

turalf commented Dec 21, 2020

/azp run python - communication - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@turalf
Copy link
Contributor Author

turalf commented Dec 21, 2020

/azp run python - communication - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@turalf turalf changed the title Remove redundant decorators; skip managed identity test [Communication] Fix async identity client decode - url bug Dec 21, 2020
Copy link
Contributor

@jbeauregardb jbeauregardb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the bug fix, didn't have idea of this

@jbeauregardb jbeauregardb merged commit e43995b into Azure:master Dec 22, 2020
rakshith91 pushed a commit to rakshith91/azure-sdk-for-python that referenced this pull request Jan 8, 2021
* Remove redundant decorators; skip managed identity test

* add isasync optional param for get auth policy

* Enable async managed identity test

* Force decoding url in async client

Co-authored-by: turalf <tufarhad@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants