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

Implement synchronous interactive authentication #6466

Merged
merged 4 commits into from
Jul 30, 2019

Conversation

chlowell
Copy link
Member

Closes #6350 with a credential for synchronous interactive authentication via the authorization code flow (an async equivalent is tracked by #6451). When this credential's get_token method is called, it opens a browser for the user to authenticate with AAD, stands up a local HTTP server and waits to receive the post-auth redirect, honoring an optional timeout. Testing this live is obviously tricky, so only offline tests are included here.

@chlowell chlowell added Client This issue points to a problem in the data-plane of the library. Azure.Identity labels Jul 23, 2019
@chlowell chlowell requested a review from schaabs July 23, 2019 23:36
@chlowell chlowell self-assigned this Jul 23, 2019
@adxsdk6
Copy link

adxsdk6 commented Jul 23, 2019

Can one of the admins verify this patch?

@chlowell
Copy link
Member Author

/azp run python - client - ci

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@chlowell
Copy link
Member Author

/azp run python - client - ci

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@chlowell chlowell merged commit f9f2db1 into Azure:master Jul 30, 2019
@chlowell chlowell deleted the interactive branch July 30, 2019 21:04
xiangyan99 added a commit that referenced this pull request Aug 5, 2019
* document async transport requirement (#6541)

* [AutoPR] alertsmanagement/resource-manager (#5697)

* Generated from f4b30b3785ae1fa51f4333f2acd863137d9c84b2 (#5696)

removed "required" field from ExpandDetectorParameter

* Packaging update of azure-mgmt-alertsmanagement

* Packaging update of azure-mgmt-alertsmanagement

* regenerated package

* recent version

* update history and version... again...

* added note about general breaking changes

* Synchronous username/password auth (#6416)

* Synchronous interactive browser authentication (#6466)

* we dont need thread locks (#6551)

* KV aiohttp by default (#6563)

* Aiohttp default async transport

* Record keys with aiohttp

* Record secrets with aiohttp

* [AutoPR hanaonazure/resource-manager] Removing monitoring hana instance API (#6535)

* Generated from 70b51057c4cca0cdb6a743b4e02f39c132121570

Removing monitoring hana instance API

* Generated from a6757c439ed37f69147b6a7b50a20f1ca69789be

Removing monitoring hana instance API

* Packaging update of azure-mgmt-hanaonazure

* Updated Release History

* KV moved paging return type to ItemPaged  (#6558)

* KV with latest autorest

* Make KV return ItemPaged from azure-core

* azure-core history 1.0.0b2 (#6562)

* azure-core history

* Update HISTORY.md

* Make private Cosmos modules private [WIP] (#6329)

* make consistent_hash_ring private

* make default_retry_policy private

* make endpoint_discovery_retry_policy private

* make hash_partition_resolver private

* make location_cache private

* make murmur_hash private

* make range private

* make range_partition_resolver private

* make vector_session_token private

* make resource_throttle_retry_policy private

* make retry_utility private

* make utils private

* make routing private

* make execution_context private

* make cosmos_client_connection private

* make retry_options private

* make query_iterable private

* make constants private

* make synchronized_request private

* make session_retry_policy private

* make partition private

* make global_endpoint_manager private

* make runtime_constants private

* make session private

* make request_object private

* make base private

* Accept extension of JSON content-type (#6583)

* Accept extension of JSON content-type

* Adding text/something+json test

* Support digit

* Remove docdb mgmt package from master (#6585)

* Revert "Remove docdb mgmt package from master (#6585)" (#6593)

This reverts commit d2ed7d8.

* azure-core black/pylint/mypy (#6581)

* Pylint/black happyness

* mypy

* Fix exceptiont tests

* bad-continuation ignroe by defulat

* adjusting to allow default omission of packages for CI. (#6595)

* Synchronous device code credential (#6464)

* [AutoPR alertsmanagement/resource-manager] fixing subscription id issue (#6574)

* Generated from d6b1b67df4f4ddc0b8cb16095d1dd51305a078a5

fix subscriptionId

* Generated from 10bf6ed3f7f87035d0c08c83742eba0a39602b39

undone reference change

* added changelog, upgraded to rc2

* fixed code generation issues

* Remove Configuration from public API (#6603)

* [AutoPR] security/resource-manager (#5709)

* Generated from 2b4c25b67ef444e5fb6df8a4a4d78bfa747b198a (#5704)

chore: jsonfmt security

Ran `jsonfmt -w "specification/security/**/*.json"`

* [AutoPR security/resource-manager] Changes to the JIT API for Firewall Support (#6047)

* Generated from 0a90b13aa3eeec05f438be7bbe0eb444a084e789

Update jitNetworkAccessPolicies.json

* Generated from 77db60c613fff13703dc99ca4aaa81bfbd4900ef

Update jitNetworkAccessPolicies.json

* Generated from 77db60c613fff13703dc99ca4aaa81bfbd4900ef

Update jitNetworkAccessPolicies.json

* Packaging update of azure-mgmt-security

* [AutoPR security/resource-manager] Improving Security Center pricing API documentation (#6289)

* Generated from 149ce8a94fd12df4868cb641d0d47bf83472b659

Improving Security Center pricing API documentation

* Generated from 149ce8a94fd12df4868cb641d0d47bf83472b659

Improving Security Center pricing API documentation

* [AutoPR security/resource-manager] Add Microsoft.Security applicationWhitelistings API (#6262)

* Generated from 0bc0b6465c2c93267a29e4eb0c39e2b12a8b623e

add operation ID's

* Generated from 56fe4277e879002f98abc961fc5fb3abf80cf32a

remove location from example, add Executable to fileType

* Generated from 6307743902cd21e4a200cca1cddc2383121ae216

add Whitelistings to custom-words

* Generated from 01357ba2f4e4ca899217b4e65892b7bdf375d5bf (#6515)

chore: jsonfmt security

* [AutoPR security/resource-manager] Set IoTSecuritySolutions & IoTSecuritySolutionAnalytics API as stable (#6526)

* Generated from 4d6fbc52713c0423aa04f74ae330867fac488b86

Set IoTSecuritySolutions & IoTSecuritySolutionAnalytics API as stable

* Generated from e763a96421ef60d69bbae038d008c08002f16359

remove paging support from IoTSecurityDeviceRecommendationsList

* Generated from dea80a5e1da4b383480342bba4eca97ba55c053b

remove paging from IoTSecurityAlertedDevicesList

* regenerated

* history and version

* general breaking changes info

* Minimal change to disable code coverage publishing for PRs. (#6614)

* Readme doc update for azure-core (#6611)

* Readme doc update

* More readme update

* Transport update

* Typo

* typo thanks bryan

* Mypy fixes for azure.core.tracing (#6590)

* we dont need thread locks

* add contextvars

* fix version stuff

* mypy fixes

* added test for get_name and moved stuff in the function

* small ci fix

* more mypy fixes

* get rid of dependency

* all use same context

* MyPy azure-core (#6619)

* MyPy azure-core settings

* MyPy azure-core aiohttp

* Redesign sentinel

* Update Key Vault docstrings (#6632)

* Update Key Vault user agent (#6640)

* Update README.md (#6635)

* mypy fixes (#6641)

* mypy fixes

* use six

* Policies as kwargs for KeyVault (#6616)

* Mypy fixes (#6646)

* [AutoPR healthcareapis/resource-manager] Fixed healthcareapi readme.md (#6652)

* Generated from 06bfc11dfa82fb5c140c82c3496fd59adc6f3b8b

just triggereing rebuild

* Packaging update of azure-mgmt-healthcareapis

* fixed release date

* fixed bug in maxItemCount propagation for Order by queries (#6608)

* fixed bug with maxItemCount in order by queries

* fixed merge conflicts

* Final azure-identity preview 2 changes (#6664)
xiangyan99 added a commit that referenced this pull request Aug 5, 2019
* App config code review feedback (#6647)

* Use new configuration

* Put version into version.py

* update packaging

* remove readme.rst

* get latest master (#6665)

* document async transport requirement (#6541)

* [AutoPR] alertsmanagement/resource-manager (#5697)

* Generated from f4b30b3785ae1fa51f4333f2acd863137d9c84b2 (#5696)

removed "required" field from ExpandDetectorParameter

* Packaging update of azure-mgmt-alertsmanagement

* Packaging update of azure-mgmt-alertsmanagement

* regenerated package

* recent version

* update history and version... again...

* added note about general breaking changes

* Synchronous username/password auth (#6416)

* Synchronous interactive browser authentication (#6466)

* we dont need thread locks (#6551)

* KV aiohttp by default (#6563)

* Aiohttp default async transport

* Record keys with aiohttp

* Record secrets with aiohttp

* [AutoPR hanaonazure/resource-manager] Removing monitoring hana instance API (#6535)

* Generated from 70b51057c4cca0cdb6a743b4e02f39c132121570

Removing monitoring hana instance API

* Generated from a6757c439ed37f69147b6a7b50a20f1ca69789be

Removing monitoring hana instance API

* Packaging update of azure-mgmt-hanaonazure

* Updated Release History

* KV moved paging return type to ItemPaged  (#6558)

* KV with latest autorest

* Make KV return ItemPaged from azure-core

* azure-core history 1.0.0b2 (#6562)

* azure-core history

* Update HISTORY.md

* Make private Cosmos modules private [WIP] (#6329)

* make consistent_hash_ring private

* make default_retry_policy private

* make endpoint_discovery_retry_policy private

* make hash_partition_resolver private

* make location_cache private

* make murmur_hash private

* make range private

* make range_partition_resolver private

* make vector_session_token private

* make resource_throttle_retry_policy private

* make retry_utility private

* make utils private

* make routing private

* make execution_context private

* make cosmos_client_connection private

* make retry_options private

* make query_iterable private

* make constants private

* make synchronized_request private

* make session_retry_policy private

* make partition private

* make global_endpoint_manager private

* make runtime_constants private

* make session private

* make request_object private

* make base private

* Accept extension of JSON content-type (#6583)

* Accept extension of JSON content-type

* Adding text/something+json test

* Support digit

* Remove docdb mgmt package from master (#6585)

* Revert "Remove docdb mgmt package from master (#6585)" (#6593)

This reverts commit d2ed7d8.

* azure-core black/pylint/mypy (#6581)

* Pylint/black happyness

* mypy

* Fix exceptiont tests

* bad-continuation ignroe by defulat

* adjusting to allow default omission of packages for CI. (#6595)

* Synchronous device code credential (#6464)

* [AutoPR alertsmanagement/resource-manager] fixing subscription id issue (#6574)

* Generated from d6b1b67df4f4ddc0b8cb16095d1dd51305a078a5

fix subscriptionId

* Generated from 10bf6ed3f7f87035d0c08c83742eba0a39602b39

undone reference change

* added changelog, upgraded to rc2

* fixed code generation issues

* Remove Configuration from public API (#6603)

* [AutoPR] security/resource-manager (#5709)

* Generated from 2b4c25b67ef444e5fb6df8a4a4d78bfa747b198a (#5704)

chore: jsonfmt security

Ran `jsonfmt -w "specification/security/**/*.json"`

* [AutoPR security/resource-manager] Changes to the JIT API for Firewall Support (#6047)

* Generated from 0a90b13aa3eeec05f438be7bbe0eb444a084e789

Update jitNetworkAccessPolicies.json

* Generated from 77db60c613fff13703dc99ca4aaa81bfbd4900ef

Update jitNetworkAccessPolicies.json

* Generated from 77db60c613fff13703dc99ca4aaa81bfbd4900ef

Update jitNetworkAccessPolicies.json

* Packaging update of azure-mgmt-security

* [AutoPR security/resource-manager] Improving Security Center pricing API documentation (#6289)

* Generated from 149ce8a94fd12df4868cb641d0d47bf83472b659

Improving Security Center pricing API documentation

* Generated from 149ce8a94fd12df4868cb641d0d47bf83472b659

Improving Security Center pricing API documentation

* [AutoPR security/resource-manager] Add Microsoft.Security applicationWhitelistings API (#6262)

* Generated from 0bc0b6465c2c93267a29e4eb0c39e2b12a8b623e

add operation ID's

* Generated from 56fe4277e879002f98abc961fc5fb3abf80cf32a

remove location from example, add Executable to fileType

* Generated from 6307743902cd21e4a200cca1cddc2383121ae216

add Whitelistings to custom-words

* Generated from 01357ba2f4e4ca899217b4e65892b7bdf375d5bf (#6515)

chore: jsonfmt security

* [AutoPR security/resource-manager] Set IoTSecuritySolutions & IoTSecuritySolutionAnalytics API as stable (#6526)

* Generated from 4d6fbc52713c0423aa04f74ae330867fac488b86

Set IoTSecuritySolutions & IoTSecuritySolutionAnalytics API as stable

* Generated from e763a96421ef60d69bbae038d008c08002f16359

remove paging support from IoTSecurityDeviceRecommendationsList

* Generated from dea80a5e1da4b383480342bba4eca97ba55c053b

remove paging from IoTSecurityAlertedDevicesList

* regenerated

* history and version

* general breaking changes info

* Minimal change to disable code coverage publishing for PRs. (#6614)

* Readme doc update for azure-core (#6611)

* Readme doc update

* More readme update

* Transport update

* Typo

* typo thanks bryan

* Mypy fixes for azure.core.tracing (#6590)

* we dont need thread locks

* add contextvars

* fix version stuff

* mypy fixes

* added test for get_name and moved stuff in the function

* small ci fix

* more mypy fixes

* get rid of dependency

* all use same context

* MyPy azure-core (#6619)

* MyPy azure-core settings

* MyPy azure-core aiohttp

* Redesign sentinel

* Update Key Vault docstrings (#6632)

* Update Key Vault user agent (#6640)

* Update README.md (#6635)

* mypy fixes (#6641)

* mypy fixes

* use six

* Policies as kwargs for KeyVault (#6616)

* Mypy fixes (#6646)

* [AutoPR healthcareapis/resource-manager] Fixed healthcareapi readme.md (#6652)

* Generated from 06bfc11dfa82fb5c140c82c3496fd59adc6f3b8b

just triggereing rebuild

* Packaging update of azure-mgmt-healthcareapis

* fixed release date

* fixed bug in maxItemCount propagation for Order by queries (#6608)

* fixed bug with maxItemCount in order by queries

* fixed merge conflicts

* Final azure-identity preview 2 changes (#6664)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Azure.Identity Client This issue points to a problem in the data-plane of the library.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Identity: Implement Interactive Authentication Support
3 participants