Skip to content

PHPC-1180: Fix STANDALONE_SSL build failures on Travis #962

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 7 commits into from
Feb 28, 2019

Conversation

jmikola
Copy link
Member

@jmikola jmikola commented Feb 28, 2019

The --user option does not solicit a value, so this was previously attempting to install a package sharing a name with the current user.
This was originally added in 0e8cf55 but should have been removed in 3678fa5.
These were originally added in mongodb-labs/mongo-orchestration@4f71d29, but it's not clear what they were used for since server configs specify their own paths and Mongo Orchestration relies on MONGO_ORCHESTRATION_HOME.
This will allow Mongo Orchestration to connect to a server/cluster using SSL.
sslWeakCertificateValidation is more permissive, so this may be redundant. MO's own configurations don't specify this for SSL environments.
This option was introduced in 3.0 and supersedes sslWeakCertificateValidation.
Per my original comment in the ticket, this URI option could conflict with older SSL-specific tests that relied on peer verification using PEM files. We'll still need to investigate those tests later, but it turns out this option wasn't needed for other tests to connect to an SSL server as previously assumed.
@jmikola jmikola merged commit a066597 into mongodb:master Feb 28, 2019
jmikola added a commit that referenced this pull request Feb 28, 2019
@jmikola jmikola deleted the phpc-1180 branch February 28, 2019 01:06
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.

1 participant