Skip to content

Commit

Permalink
E-mail integration and AWS KMS in task runner:
Browse files Browse the repository at this point in the history
  • Loading branch information
Ardem committed Sep 24, 2024
1 parent 35049d1 commit 2c7d749
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 8 deletions.
1 change: 1 addition & 0 deletions .env.common
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ YLEM_LOG_LEVEL=trace
# To enable encrypting and decrypting your organization, user and integration data with AWS KMS,
# you need to place your AWS credentials here
# More information: https://docs.ylem.co/open-source-edition/configuring-integrations-with-.env-variables#using-aws-kms-to-encrypt-decrypt-data
# Use same variables for sending E-mails with Amazon SES
AWS_KMS_KEY_ID=
AWS_REGION=
AWS_ACCESS_KEY_ID=
Expand Down
6 changes: 0 additions & 6 deletions processor/taskrunner/.env
Original file line number Diff line number Diff line change
Expand Up @@ -6,12 +6,6 @@ TASK_RUNNER_GOPYK_BASE_URL=http://ylem_python_processor:7338/eval
# By default it assumes that it is running on the port 7890 on your host machine
TASK_RUNNER_TABLEAU_HTTP_WRAPPER_BASE_URL=http://host.docker.internal:7890

TASK_RUNNER_SLACK_CLIENT_ID=
TASK_RUNNER_SLACK_CLIENT_SECRET=

TASK_RUNNER_AWS_KMS_KEY_ID=
TASK_RUNNER_AWS_REGION=

# To enable ChatGPT integration, create its API secret key
# And place it here
# More information: https://docs.ylem.co/pipelines/tasks-ip/gpt
Expand Down
4 changes: 2 additions & 2 deletions processor/taskrunner/config/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,8 @@ type kafka struct {
}

type aws struct {
Region string `split_words:"true" default:"eu-central-1"`
KmsKeyId string `split_words:"true"`
Region string `split_words:"true" default:"eu-central-1" envconfig:"AWS_REGION"`
KmsKeyId string `split_words:"true" envconfig:"AWS_KMS_KEY_ID"`
}

type twilio struct {
Expand Down

0 comments on commit 2c7d749

Please sign in to comment.