[Lock] MongoDbStore deprecated extracting database/collection from MongoDB connection URI

[kralos]

Q	A
Branch?	master
Bug fix?	no
New feature?	no
Deprecations?	yes
Tickets	Fix #37180
License	MIT
Doc PR

We should not overload the MongoDB connection URI with a non-standard querysting like collection.

The /path component should only be used for the authentication database, never the default database.

see https://docs.mongodb.com/manual/reference/connection-string/

see #37140 (comment)

[nicolas-grekas]

Being able to take full control over the connection setting with a DSN is desired to me. But this doesn't mean this DSN string should be passed as is to the underlying mongo connection object. Instead, we should extract these extra settings and filter them from the uri passed to mongo. Does this make sense?

[kralos]

Instead, we should extract these extra settings and filter them from the uri passed to mongo

could do

[fabpot]

@kralos Is it still a draft? /cc @jderusse

[kralos]

@kralos Is it still a draft? /cc @jderusse

No, I think we should close this / don't merge. This was based on jmikola's concerns around non-standard options being part of the mongodb connection string (database and collection). However #37180 provides a middle ground allowing uri only to control down to the collection level without passing custom parameters down to the driver.

Since that is now merged, I'm now closing this ticket.