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

Adding documentation for db_config file #90

Closed
wants to merge 1 commit into from

Conversation

nszchagas
Copy link

What was done

  • Included documentation for database configuration file structure.

@trasher
Copy link
Collaborator

trasher commented Sep 19, 2023

Database configuration file is automatically generated; there is no need to document it.

Some part of that file are automatically calculated; this is really not a good idea to try changing this file.

I'd vote against the integration of this PR.

@nszchagas
Copy link
Author

Even though it's automatically generated, it's important to know where it is stored.

I'm running GLPI on a Docker container using a personalized docker image, and there's the need to bind this configuration file on my server, in order to protect my database variables, since the image is shared in containers registries.

@trasher
Copy link
Collaborator

trasher commented Sep 19, 2023

Path is already present in the advanced installation part, but its does not use the constant - that must be changed to be consistent across the doc.

Path is quite important, yes, so just keep that information, and remove file contents.

@nszchagas nszchagas closed this by deleting the head repository Mar 17, 2024
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.

None yet

2 participants