Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: apply recommneded settings for running open edx proxy #46

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ghassanmas
Copy link
Contributor

This relate to the infitie redirct issue that comes eveyr now and then in Chrome:;

According to tutor docs those are the correct/expected config when running behind a proxy.

@snglth since you have changed the config before, can you take a look into this change

@snglth
Copy link
Member

snglth commented Oct 4, 2023

This is basically how community.abzt.de functions now. Those settings are being set by ansible. https://github.com/Abstract-Tech/community-theme-tutor/blob/main/ansible/group_vars/tutor/config.yml#L6

@ghassanmas
Copy link
Contributor Author

@snglth those might changes when I was debugging the issue, if I rerun the CI/CD, those settings should be there, right?

@snglth
Copy link
Member

snglth commented Oct 4, 2023

Yes

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.

2 participants