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

DB auth has problems with the new dotenv config (fix included) #847

Closed
jasonmunro opened this issue Dec 21, 2023 · 1 comment · Fixed by #852
Closed

DB auth has problems with the new dotenv config (fix included) #847

jasonmunro opened this issue Dec 21, 2023 · 1 comment · Fixed by #852
Assignees

Comments

@jasonmunro
Copy link
Member

Found and fixed an issue where my dotenv values were not being used for database authentication. Just a naming mismatch in the new config code. The following fixes the issue:

diff --git a/config/database.php b/config/database.php
index c7ee2187..2e6a1499 100644
--- a/config/database.php
+++ b/config/database.php
@@ -17,7 +17,7 @@ return [
     | Connection type. Can be "host" to connect to a hostname, or "socket" to
     | connect to a unix socket.
     */
-    'db_conn_type' => env('DB_CONNECTION_TYPE', 'host'),
+    'db_connection_type' => env('DB_CONNECTION_TYPE', 'host'),

     /*
     |
@@ -63,5 +63,5 @@ return [
     |
     | Database type. can be any supported PDO driver ; (http://php.net/manual/en/pdo.drivers.php)
     */
-    'db_driver' => env('DRIVER','mysql')
+    'db_driver' => env('DB_DRIVER','mysql')
 ];
@Shadow243
Copy link
Member

PR: #852

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 a pull request may close this issue.

2 participants