Skip to content

add CSRF_TRUSTED_ORIGINS#320

Merged
GDay merged 2 commits intochiefonboarding:masterfrom
IamTaoChen:master
Jun 18, 2023
Merged

add CSRF_TRUSTED_ORIGINS#320
GDay merged 2 commits intochiefonboarding:masterfrom
IamTaoChen:master

Conversation

@IamTaoChen
Copy link
Copy Markdown
Contributor

No description provided.

@GDay
Copy link
Copy Markdown
Member

GDay commented Jun 18, 2023

Could you update it to use a list (env.list) instead? it will automatically split on , and you can add a default [] to make it a one line thing :)
https://django-environ.readthedocs.io/en/latest/api.html#environ.Env.list

Thanks for the PR!

@coveralls
Copy link
Copy Markdown
Collaborator

Coverage Status

coverage: 93.03% (-0.02%) from 93.046% when pulling 019f034 on TaoChan1005:master into 3d56c89 on chiefonboarding:master.

@GDay GDay self-requested a review June 18, 2023 01:21
Copy link
Copy Markdown
Member

@GDay GDay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks!

@GDay GDay merged commit 9654228 into chiefonboarding:master Jun 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants