Skip to content

feature/gha #1

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

Closed
wants to merge 5 commits into from
Closed

feature/gha #1

wants to merge 5 commits into from

Conversation

OskarStark
Copy link
Owner

OskarStark and others added 5 commits December 26, 2022 21:08
* Use literals instead of RC/RP/WC constants for conciseness

* PHPLIB-1057: Note that geoHaystack was removed in MongoDB 5.0+

* Use correct boolean type in updateMany() example code

* Better clarify Database::command() purpose and examples

Also fixes two long-standing syntax errors in the example code

* Revise command tutorial and sync with command() docs

Remove the geoNear example since the command itself was removed in MongoDB 4.2.
)

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

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).

* Fix static method invocations

* Append TLS options to LB URIs if necessary
It is still used here and there in the CI
@OskarStark OskarStark self-assigned this Jan 10, 2023
@OskarStark OskarStark closed this Jan 13, 2023
@OskarStark OskarStark deleted the feature/gha branch January 13, 2023 08:31
OskarStark pushed a commit that referenced this pull request Feb 28, 2025
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