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

fix: get good project_id from jwt or google-creds when they are set or not [TCTC-6942] #1308

Merged
merged 2 commits into from
Oct 3, 2023

Conversation

Sanix-Darker
Copy link
Contributor

@Sanix-Darker Sanix-Darker commented Oct 2, 2023

WHAT

get the "good" and "valid" project-id from the connector configuration

WHY

When both JWT and GoogleCreds were sets, on the "configuration for listing tables and schemas", we were using the wrong project-id to make requests, now, we make sure the jwt-credentials is checkec but also contains all informations needed.

HOW

  • fix: fallback when the project-id is not well get from either the jwt_credentials or the google-credentials
  • add some Unit tests to illustrate that
  • doc: update the CHANGELOG.md

ADDITIONAL INFOS

this branch is deployed on the ange guru and can be tested on the app TCTC-6942

@Sanix-Darker Sanix-Darker self-assigned this Oct 2, 2023
@Sanix-Darker Sanix-Darker force-pushed the fix/gbq-configuration branch from f4f4d45 to e4453a2 Compare October 2, 2023 14:04
…_credentials or the google-credentials

doc: update the CHANGELOG.md
@Sanix-Darker Sanix-Darker force-pushed the fix/gbq-configuration branch from e4453a2 to c917f0d Compare October 3, 2023 00:31
@Sanix-Darker Sanix-Darker marked this pull request as ready for review October 3, 2023 00:52
@Sanix-Darker Sanix-Darker force-pushed the fix/gbq-configuration branch from 76c189f to 398f6ef Compare October 3, 2023 01:10
@Sanix-Darker Sanix-Darker changed the title fix: consistantly get project_id from jwt or google-creds when they are set or not [TCTC-6942] fix: get good project_id from jwt or google-creds when they are set or not [TCTC-6942] Oct 3, 2023
@Sanix-Darker Sanix-Darker requested a review from aklom October 3, 2023 06:45
Copy link
Contributor

@aklom aklom left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@Sanix-Darker Sanix-Darker merged commit 78cbf38 into master Oct 3, 2023
3 checks passed
@Sanix-Darker Sanix-Darker deleted the fix/gbq-configuration branch October 3, 2023 22:39
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