backup-manager / laravel

Driver to seamlessly integrate the Backup Manager into Laravel applications.
MIT License
655 stars 149 forks source link

Crash when database connection is via socket (so no host) #128

Open brucek2 opened 4 years ago

brucek2 commented 4 years ago

It looks like src/GetDatabaseConfig:34 assumes there will be a 'host' index defined in the $connection array, although this will not be the case for a connection via socket vs. TCP/IP.

This is a bummer because it means if I want backup-manager/laravel installed I am forced to use the higher overhead TCP/IP connection for all of my application, even though the socket connection works fine for the rest of my Laravel app when backup-manager/laravel is not installed.