Skip to content
This repository was archived by the owner on Mar 17, 2025. It is now read-only.

wip: parse mongo uri#276

Open
germanafro wants to merge 8 commits intomrvautin:masterfrom
germanafro:master
Open

wip: parse mongo uri#276
germanafro wants to merge 8 commits intomrvautin:masterfrom
germanafro:master

Conversation

@germanafro
Copy link

the 'mongo-uri' component is 6 years old and outdated so replaced it

TODO:
catch invalid URIs and throw errors

…ct to admin db when calling index with no db

+ added authDatabase field to MongoUri that will default to 'admin'
+ index now calls get_db_stats with authDatabase
+ index now calls get_sidebar_list with authDatabase
… bug wa sit being '' when '/' was present at the end of connect string. no authDb logic required
@ZhaoKunLong
Copy link

@germanafro I also got that problem. Can you add the reviewers? Maybe cause you don't add it, so they are not got notifications.

@germanafro
Copy link
Author

@mrvautin

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants