Skip to content

PHPC-1156: Prefer Secure Transport over OpenSSL with --with-mongodb-ssl on Darwin #795

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 1 commit into from
Apr 11, 2018

Conversation

derickr
Copy link
Contributor

@derickr derickr commented Apr 10, 2018

https://jira.mongodb.org/browse/PHPC-1156

@kvwalker — as I don't have a Mac, could you test this patch by running ./configure --with-mongodb-ssl and see if it picks Secure Transport please? Before the patch it should (try to) pick OpenSSL.

@jmikola jmikola requested a review from kvwalker April 10, 2018 16:12
@jmikola
Copy link
Member

jmikola commented Apr 10, 2018

Should just require running ./configure as --with-mongodb-ssl already defaults to auto.

@kvwalker
Copy link
Contributor

I think this works! Before the change, my Mac shows
Build configuration:
CFLAGS : -g -O2
...
MONGODB_SHARED_LIBADD : -lsasl2 -lssl -lcrypto -lz -lz

After the change, it shows
Build configuration:
CFLAGS : -g -O2
...
MONGODB_SHARED_LIBADD : -lsasl2 -framework Security -framework CoreFoundation -lz

@jmikola jmikola self-requested a review April 10, 2018 20:03
Copy link

@vntw vntw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Failed previously but works now perfectly after applying the patch 🎉

@derickr derickr force-pushed the PHPC-1156-darwin-secure-transport branch from 7511100 to cb81027 Compare April 11, 2018 09:36
@derickr derickr merged commit cb81027 into mongodb:v1.4 Apr 11, 2018
derickr added a commit that referenced this pull request Apr 11, 2018
@jmikola
Copy link
Member

jmikola commented Apr 11, 2018

@derickr: For context, @venyii is @alcaeus' co-worker who previously experienced some issues with OpenSSL on macOS.

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.

4 participants