Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Update GcpCredentials to use SecretDict for service_account_info #88

Merged
merged 18 commits into from
Dec 20, 2022

Conversation

ahuang11
Copy link
Contributor

@ahuang11 ahuang11 commented Dec 16, 2022

Closes #73 by making use of SecretDict to obfuscate GcpCredentials.service_account_info. This is a breaking change.

Example

Screenshots

Checklist

  • References any related issue by including "Closes #" or "Closes ".
    • If no issue exists and your change is not a small fix, please create an issue first.
  • Includes tests or only affects documentation.
  • Passes pre-commit checks.
    • Run pre-commit install && pre-commit run --all locally for formatting and linting.
  • Includes screenshots of documentation updates.
    • Run mkdocs serve view documentation locally.
  • Summarizes PR's changes in CHANGELOG.md

@ahuang11 ahuang11 marked this pull request as ready for review December 17, 2022 00:08
@ahuang11 ahuang11 changed the title Update credentials Update GcpCredentials to use SecretDict for service_account_info Dec 17, 2022
desertaxle
desertaxle previously approved these changes Dec 19, 2022
@ahuang11
Copy link
Contributor Author

This works now:
image

@ahuang11 ahuang11 merged commit 907fc32 into main Dec 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make service_account_info on the GcpCredentials block a SecretStr type
2 participants