Skip to content

Conversation

@JonathanLeeWH
Copy link
Contributor

@JonathanLeeWH JonathanLeeWH commented Nov 25, 2020

  • Refactored firebaseConfig.js to use environmental variables instead of service account json file
  • Amended dockerfile, dockerfile-ci, docker-compose.yml with the new firebase admin service account environment variables
  • Removed the encrypted secret firebase.gpg, the job for decryption of encrypted secrets, the decrypt_firebase_secret.sh shell script
  • Removed GOOGLE_APPLICATION_CREDENTIALS environment variable which is no longer used
  • Improved Readme.md documentation with the new firebase service account environment variables
  • Refactored TEST_FIREBASE_CLIENT_API_KEY environment variables to FIREBASE_CLIENT_API_KEY as it is currently used not just for testing purposes

Resolved #81

Amended `dockerfile`, `dockerfile-ci,` `docker-compose.yml` with the new firebase admin service account environment variables
Removed the encrypted secret `firebase.gpg`, the job for decryption of encrypted secrets, the `decrypt_firebase_secret.sh` shell script
Removed `GOOGLE_APPLICATION_CREDENTIALS` environment variable which is no longer used
@JonathanLeeWH JonathanLeeWH added github_actions Pull requests that update Github_actions code Security Related to Security docker Related to docker firebase Related to firebase labels Nov 25, 2020
@JonathanLeeWH JonathanLeeWH added this to the Version 1.0.0 milestone Nov 25, 2020
@JonathanLeeWH JonathanLeeWH self-assigned this Nov 25, 2020
@JonathanLeeWH JonathanLeeWH changed the title Switch GitHub Action Continuous Integration Approach to environment variables Improved handling Google Service Account Approach in GitHub Actions Nov 25, 2020
@JonathanLeeWH JonathanLeeWH added priority.high High priority documentation Improvements or additions to documentation labels Nov 25, 2020
Refactored `TEST_FIREBASE_CLIENT_API_KEY ` environment variables to `FIREBASE_CLIENT_API_KEY` as it is currently used not just for testing purposes
@JonathanLeeWH JonathanLeeWH added the status.done Done and ready for review and merge label Nov 25, 2020
@JonathanLeeWH JonathanLeeWH merged commit 985aa4a into master Nov 25, 2020
@JonathanLeeWH JonathanLeeWH deleted the refactor-firebase-credentials branch November 25, 2020 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docker Related to docker documentation Improvements or additions to documentation firebase Related to firebase github_actions Pull requests that update Github_actions code priority.high High priority Security Related to Security status.done Done and ready for review and merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improved handling Google Service Account Approach in GitHub Actions ci

2 participants