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
feat: Make full implementation on DownscopedClient. #1232
feat: Make full implementation on DownscopedClient. #1232
Conversation
Seems like this code was copied from other classes. I remember a thread in the cab doc that mentioned a refactor. Did that change? |
There're 2 approaches copy vs refactor. In short term, it is easier to just copy from baseExternalClient. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Just one test missing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we figure out a way to make base external client and downscoped client share logic?
🤖 I have created a release \*beep\* \*boop\* --- ## [7.9.0](https://www.github.com/googleapis/google-auth-library-nodejs/compare/v7.8.0...v7.9.0) (2021-09-02) ### Features * wire up implementation of DownscopedClient. ([#1232](https://www.github.com/googleapis/google-auth-library-nodejs/issues/1232)) ([baa1318](https://www.github.com/googleapis/google-auth-library-nodejs/commit/baa13185836bb299b769c034bfb7d37231eea8d1)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
This full implementation for downscoped client is just to satisfy requirement of the abstract class AuthClient, this is neither urgent and nor a blocker in terms of functionality since they're not expected to be used by token brokers.
Three components are added:
These implementations are mostly copied from BaseExternalClient.