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

🐛 Bug Report: Appwrite Console Whitelist Root is disabled by default #2648

Open
2 tasks done
Exadra37 opened this issue Jan 19, 2022 · 3 comments
Open
2 tasks done
Assignees
Labels
bug Something isn't working

Comments

@Exadra37
Copy link

👟 Reproduction steps

In the config/variables.php says that by default is enabled, but in the .env file is disabled.

👍 Expected behavior

I would expect to see enabled in the .env file.

👎 Actual Behavior

It's configured as disabled in the .env file.

🎲 Appwrite version

Version 0.12.x

💻 Operating system

Linux

🧱 Your Environment

No.

👀 Have you spent some time to check if this issue has been raised before?

  • I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

@Exadra37 Exadra37 added the bug Something isn't working label Jan 19, 2022
@eldadfux
Copy link
Member

This .env file is used only for Appwrite maintainers and is not part of the installation script on instructions. The file use for installation is generated by this config: https://github.com/appwrite/appwrite/blob/master/app/config/variables.php and the recommended docker-compose manual setup is located here: https://gist.github.com/eldadfux/977869ff6bdd7312adfd4e629ee15cc5

@Exadra37
Copy link
Author

Thanks for the clarification :)

To be honest I find it confusing that the docker-compose file in the repo is not meant for a production setup (after properly configured via .env), but I understand the Appwrite team motivation to use as it is.

Maybe adding a docker-compos.prod.yml file to the repo with the gist setup would avoid others to get confused like myself?

@eldadfux eldadfux self-assigned this Jan 30, 2022
@eldadfux eldadfux assigned TorstenDittmann and unassigned eldadfux Mar 20, 2022
@eldadfux
Copy link
Member

@TorstenDittmann maybe we can pre-generate the prod file as part of the improvements to the setup script.

@eldadfux eldadfux reopened this Mar 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants