Skip to content

PHPLIB-794: Use single/multi-mongos LB URIs when available #1019

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

Merged
merged 3 commits into from
Jan 10, 2023

Conversation

jmikola
Copy link
Member

@jmikola jmikola commented Dec 23, 2022

@jmikola jmikola requested a review from alcaeus December 27, 2022 02:16
This ensures MONGODB_SINGLE_MONGOS_LB_URI and MONGODB_MULTI_MONGOS_LB_URI are now set for load balancer tasks in Evergreen and used within the test suite.

Also fixes some bugs in the original getUri() code for reassembling a URL with a single mongos host. Previously, auth credentials were not properly appended (d5b1156) and a query string could be appended without a slash between the question mark and last host in the seedlist (49d32aa).
@alcaeus alcaeus merged commit 964e887 into mongodb:master Jan 10, 2023
@jmikola jmikola deleted the phplib-794 branch January 11, 2023 16:05
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 this pull request may close these issues.

2 participants