DSNs Including DB Name

[derekrprice]

My Mongo DSN looks like mongodb://mongo:27017/dbname. Using this configuration yields an error about "$databaseName is not defined":

'mongo' => [
    'driver' => 'mongodb',
    'dsn' => 'mongodb://mongo:27017/dbname',
]

Adding a database name to the configuration works around the issue, but it is redundant with the information in the URL:

'mongo' => [
    'driver' => 'mongodb',
    'dsn' => 'mongodb://mongo:27017/dbname',
    'database' => 'dbname',
]

All our other environments in both PHP & JS work just fine with only the DSN and no DB name specified, including ones using the same base PHP Mongo driver. database should not be required with a full dsn.

[Smolevich]

it will be changed here

[derekrprice]

@divine what do the "awaiting-author" and "question" tags mean? You are waiting for someone to submit a PR?

[divine]

Hello,

Could you please confirm that this PR #1954 fixed your issue?

New release on the way.

Awaiting author means I'm waiting any more information from the author, it has been almost 5 month since last reply so not sure if it's still actual.

Thanks.

[derekrprice]

I added some comments to PR #1954. If I leave 'database' name out of my config, I still get this error:

Undefined index: database {"userId":7,"exception":"[object] (ErrorException(code: 0): Undefined index: database at /var/www/device-service/vendor/jenssegers/mongodb/src/Jenssegers/Mongodb/Connection.php:41)

This is the config that I would like to use:

'mongo' => [
    'driver' => 'mongodb',
    'dsn' => 'mongodb://mongo:27017/dbname',
]

[divine]

@derekrprice could you please just create a new PR with your proposed fixes?

I've just recreated from commit that was waiting for 2 years so you should understand and accept our apologies for my mistakes that I've made.

[derekrprice]

Yeah. I should be able to put something together within a few days.

[divine]

@derekrprice could you please check PR #1971 ?

Thanks.