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

Support OAuth authentication for Big Query #420

Closed
MonideepDe opened this issue Aug 1, 2023 · 1 comment · Fixed by #431
Closed

Support OAuth authentication for Big Query #420

MonideepDe opened this issue Aug 1, 2023 · 1 comment · Fixed by #431

Comments

@MonideepDe
Copy link
Contributor

Hi there!

Currently, cosmos only supports two authentication mechanisms:

  1. service account key file and
  2. service account key json

DBT supports oauth via gcloud authentication. See documentation here.

As part of this issue, I request the support for oauth via gcloud authentication method.

I will open a PR for the same and if everything looks good, we can hopefully merge it into main and make it a part of the next release.

Thanks!

@harels
Copy link
Contributor

harels commented Aug 1, 2023

Thanks @MonideepDe! looking forward to your PR!

@MonideepDe MonideepDe changed the title Support OAuth authentication for Biq Query Support OAuth authentication for Big Query Aug 1, 2023
jlaneve pushed a commit that referenced this issue Aug 2, 2023
## Description

Support OAuth authentication for Big Query

## Related Issue(s)
closes #420 

## Breaking Change?

None

## Checklist

- [ ] I have made corresponding changes to the documentation (if
required)
- [X] I have added tests that prove my fix is effective or that my
feature works

---------

Co-authored-by: Monideep De <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
tatiana pushed a commit that referenced this issue Aug 9, 2023
## Description

Support OAuth authentication for Big Query

## Related Issue(s)
closes #420 

## Breaking Change?

None

## Checklist

- [ ] I have made corresponding changes to the documentation (if
required)
- [X] I have added tests that prove my fix is effective or that my
feature works

---------

Co-authored-by: Monideep De <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants