Skip to content

Commit

Permalink
Add new sftp environment variables
Browse files Browse the repository at this point in the history
The SFTP service needs some additional environment variables to be
populated, as a result of some recent changes to how refresh tokens are
used to generate auth tokens [1].

A few of these vriables are redundant [2], and that's why we use the
same "source" variable to map them as late as possible in the
provisioning.  Eventually if the redundancy is removed from the sftp
service we'll want to update the provisioner to stop populating the
obsolete copies.

Issue #138

[1] PermanentOrg/sftp-service#192
  • Loading branch information
slifty committed Oct 26, 2023
1 parent 4db6b02 commit f5dc38a
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions .github/workflows/build-images.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ jobs:
notification_firebase_credentials: DEV_NOTIFICATION_FIREBASE_CREDENTIALS
fusion_auth_host_secret_key: DEV_FUSION_AUTH_HOST
fusion_auth_key_sftp_secret_key: DEV_FUSION_AUTH_KEY_SFTP
fusion_auth_sftp_app_id: DEV_FUSION_AUTH_SFTP_APP_ID
- perm_env: staging
server_domain: "staging.permanent.org"
app_id: "C8YKZNBVWT.org.permanent.permanent.staging"
Expand All @@ -44,6 +45,7 @@ jobs:
notification_firebase_credentials: STAGING_NOTIFICATION_FIREBASE_CREDENTIALS
fusion_auth_host_secret_key: STAGING_FUSION_AUTH_HOST
fusion_auth_key_sftp_secret_key: STAGING_FUSION_AUTH_KEY_SFTP
fusion_auth_sftp_app_id: STAGING_FUSION_AUTH_SFTP_APP_ID
- perm_env: prod
server_domain: "www.permanent.org"
app_id: "C8YKZNBVWT.org.permanent.PermanentArchive"
Expand All @@ -55,6 +57,7 @@ jobs:
notification_firebase_credentials: PROD_NOTIFICATION_FIREBASE_CREDENTIALS
fusion_auth_host_secret_key: PROD_FUSION_AUTH_HOST
fusion_auth_key_sftp_secret_key: PROD_FUSION_AUTH_KEY_SFTP
fusion_auth_sftp_app_id: PROD_FUSION_AUTH_SFTP_APP_ID
steps:
- uses: actions/checkout@v1
- name: Build image
Expand All @@ -79,6 +82,9 @@ jobs:
SERVER_DOMAIN: ${{ matrix.environment.server_domain }}
FUSION_AUTH_HOST: ${{ secrets[matrix.environment.fusion_auth_host_secret_key] }}
FUSION_AUTH_KEY_SFTP: ${{ secrets[matrix.environment.fusion_auth_key_sftp_secret_key] }}
FUSION_AUTH_SFTP_APP_ID: ${{ secrets[matrix.environment.fusion_auth_sftp_app_id] }}
FUSION_AUTH_SFTP_CLIENT_ID: ${{ secrets[matrix.environment.fusion_auth_sftp_app_id] }}
FUSION_AUTH_SFTP_CLIENT_SECRET: ${{ secrets[matrix.environment.fusion_auth_key_sftp_secret_key] }}
notify:
runs-on: ubuntu-20.04
needs: ["build"]
Expand Down

0 comments on commit f5dc38a

Please sign in to comment.