config: allow multiple cors origins

[taonic]

Description & motivation 💭

This config template change allows multiple CORS origins to be supplied via an comma separated env var.

Requested from: https://temporalio.slack.com/archives/CTTJCPZQE/p1692349771226649

Testing 🧪

Tested manually through:

dockerize -template ./config-template.yaml:./config/docker.yaml

How was this tested 👻

• Manual testing
• E2E tests added
• Unit tests added

Steps for others to test: 🚶🏽‍♂️🚶🏽‍♀️

Checklists

Draft Checklist

Merge Checklist

Issue(s) closed

Docs

Any docs updates needed?

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Updated (UTC)
holocene	⬜️ Ignored (Inspect)		Aug 23, 2023 11:45am

[rossedfort]

@taonic did you test specifying a single and multiple origins for this change?

[taonic]

@taonic did you test specifying a single and multiple origins for this change?

@rossedfort Yes, I should've added the test results. Better late than never.

Single:

✗ export TEMPORAL_CORS_ORIGINS=test.com
✗ dockerize -template config-template.yaml:docker.yaml
✗ grep -C 4 allowOrigins docker.yaml
batchActionsDisabled: false
hideWorkflowQueryErrors: false
cors:
  cookieInsecure: false
  allowOrigins:
    # override framework's default that allows all origins "*"
    - test.com
tls:
  caFile:

Multiple:

✗ export TEMPORAL_CORS_ORIGINS=test.com,test2.com
✗ dockerize -template config-template.yaml:docker.yaml
✗ grep -C 4 allowOrigins docker.yaml
batchActionsDisabled: false
hideWorkflowQueryErrors: false
cors:
  cookieInsecure: false
  allowOrigins:
    # override framework's default that allows all origins "*"
    - test.com
    - test2.com
tls:

Default:

✗ unset TEMPORAL_CORS_ORIGINS
✗ dockerize -template config-template.yaml:docker.yaml
✗ grep -C 4 allowOrigins docker.yaml
batchActionsDisabled: false
hideWorkflowQueryErrors: false
cors:
  cookieInsecure: false
  allowOrigins:
    # override framework's default that allows all origins "*"
    - "http://localhost:8080"
tls:
  caFile:

[karelbilek]

Also related:

temporalio/helm-charts#402