Skip to content

Conversation

@josecelano
Copy link
Member

Move installation and configuration docs to Rust Documentation Comments.

@josecelano josecelano linked an issue May 29, 2023 that may be closed by this pull request
@josecelano
Copy link
Member Author

hi @WarmBeer I do not know where the env var DATABASE_URL is used:

DATABASE_URL=sqlite://storage/database/data.db?mode=rwc

The current installation guide:

cd backend
echo "DATABASE_URL=sqlite://data.db?mode=rwc" > .env

says you have to run that. I think the .env is only used when you used docker.

Is it something that maybe you were using for the database tests and it's not used anymore?

@josecelano
Copy link
Member Author

hi @WarmBeer I do not know where the env var DATABASE_URL is used:

DATABASE_URL=sqlite://storage/database/data.db?mode=rwc

The current installation guide:

cd backend
echo "DATABASE_URL=sqlite://data.db?mode=rwc" > .env

says you have to run that. I think the .env is only used when you used docker.

Is it something that maybe you were using for the database tests and it's not used anymore?

Hey, I've just seen it's used by sqlx.

https://github.com/launchbadge/sqlx

It's a little bit confusing that the same file is used by docker-compose and sqlx but for the time being I'm just going to explain it in the documentation.

@josecelano
Copy link
Member Author

Hi @WarmBeer I think we should move another config option:

[auth]
email_on_signup = "Optional"
min_password_length = 6
max_password_length = 64
secret_key = "MaxVerstappenWC2021"

...

[mail]
email_verification_enabled = false
from = "[email protected]"
reply_to = "[email protected]"
username = ""
password = ""
server = ""
port = 25

I think w should move the email_verification_enabled from [mail] to [auth]`. If you agree I'll open a new issue.

@josecelano josecelano marked this pull request as ready for review June 5, 2023 15:53
@josecelano josecelano merged commit 29d87de into torrust:develop Jun 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Installation and configuration documentation

1 participant