Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
 into for-eduhelx-chart
  • Loading branch information
Hoid committed May 9, 2024
2 parents b240eb4 + 0c5ccbb commit c7a871a
Show file tree
Hide file tree
Showing 4 changed files with 14 additions and 1 deletion.
3 changes: 2 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,7 @@ Additionally there is a workflow that allows bumping the chart version, if this
| django.APPSTORE_DJANGO_USERNAME | string | `"admin"` | |
| django.AUTHORIZED_USERS | string | `""` | user emails for oauth providers |
| django.AUTO_WHITELIST_PATTERNS | list | `[]` | Note that these only run on a user's primary alias. If a user has [email protected] as their primary alias, and [email protected] as a secondary alias, they will only be whitelisted automatically if cs.unc.edu emails are allowed. ex. Whitelist all RENCI emails - "^[A-Za-z0-9._%+-]+@renci\\.org$" ex. Whitelist all UNC emails - "^[A-Za-z0-9._%+-]+@([A-Za-z0-9.-]+\\.)?unc\\.edu$" ex. Whitelist CS dept. (grad./prof.) UNC emails - "^[A-Za-z0-9._%+-]+@cs\\.unc\\.edu$" |
| django.CSRF_DOMAINS | string | `"https://*.renci.org,https://*.renci.unc.edu"` | allowed domains to make post requests to the appstore |
| django.DEV_PHASE | string | `"live"` | should be 'live' unless you are doing some kind of development |
| django.DOCKSTORE_APPS_BRANCH | string | `"v1.6.0"` | Specify the git branch to use for HeLx app specifications. When declaring 'tycho.externalAppRegistryRepo' leave this as an empty string. |
| django.EMAIL_HOST | string | `""` | Email Server host ie relay.unc.edu | relay.renci.org |
Expand Down Expand Up @@ -173,5 +174,5 @@ Additionally there is a workflow that allows bumping the chart version, if this
| webtop.enabled | bool | `true` | Disabling will turn off the creation of secrets/configmaps for Webtop |

----------------------------------------------
Autogenerated from chart metadata using [helm-docs v1.13.1](https://github.com/norwoodj/helm-docs/releases/v1.13.1)
Autogenerated from chart metadata using [helm-docs v1.11.0](https://github.com/norwoodj/helm-docs/releases/v1.11.0)

5 changes: 5 additions & 0 deletions templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -190,6 +190,11 @@ spec:
key: AUTHORIZED_USERS
name: {{ include "appstore.fullname" . }}
{{- end }}
- name: CSRF_DOMAINS
valueFrom:
secretKeyRef:
key: CSRF_DOMAINS
name: {{ include "appstore.fullname" . }}
- name: OAUTH_PROVIDERS
valueFrom:
secretKeyRef:
Expand Down
5 changes: 5 additions & 0 deletions templates/secrets.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,11 @@ data:
{{- else }}
OAUTH_PROVIDERS: ""
{{- end }}
{{ if .Values.django.CSRF_DOMAINS }}
CSRF_DOMAINS: {{ .Values.django.CSRF_DOMAINS | b64enc }}
{{- else }}
CSRF_DOMAINS: ""
{{- end }}
{{ if .Values.oauth.GITHUB_NAME }}
GITHUB_NAME: {{ .Values.oauth.GITHUB_NAME | b64enc }}
GITHUB_CLIENT_ID: {{ .Values.oauth.GITHUB_CLIENT_ID | b64enc }}
Expand Down
2 changes: 2 additions & 0 deletions values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -140,6 +140,8 @@ django:
RECIPIENT_EMAILS: ""
# -- should be 'live' unless you are doing some kind of development
DEV_PHASE: "live"
# -- allowed domains to make post requests to the appstore
CSRF_DOMAINS: "https://*.renci.org,https://*.renci.unc.edu"

oauth:
# -- oauth providers separated by commas (google, github)
Expand Down

0 comments on commit c7a871a

Please sign in to comment.