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.

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

Jump to bottom

[azure-mgmt-core] update ARMChallengeAuthenticationPolicy for CAE #38113

Open
msyyc opened this issue Oct 28, 2024 · 3 comments
Open

[azure-mgmt-core] update ARMChallengeAuthenticationPolicy for CAE #38113

msyyc opened this issue Oct 28, 2024 · 3 comments
Assignees
@msyyc
Labels
Mgmt This issue is related to a management-plane library.

Comments

@msyyc
Copy link
Member

msyyc commented Oct 28, 2024

Parent class BearerTokenCredentialPolicy of ARMChallengeAuthenticationPolicy already supports CAE, so ARMChallengeAuthenticationPolicy just need to inherit from it completely.
@msyyc msyyc added the Mgmt This issue is related to a management-plane library. label Oct 28, 2024
@msyyc msyyc self-assigned this Oct 28, 2024
@archerzz
Copy link
Member

@msyyc Can you add a mock test? Thus, I don't need to file a separate issue. Thanks.

@msyyc
Copy link
Member Author

msyyc commented Nov 14, 2024

We could refer test cases of other languages:

@msyyc
Copy link
Member Author

msyyc commented Nov 15, 2024

@msyyc Can you add a mock test? Thus, I don't need to file a separate issue. Thanks.

Sure, will do it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@msyyc msyyc

Labels
Mgmt This issue is related to a management-plane library.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@archerzz @msyyc