Fix ssl verify

[JasonGiedymin]

The original code did not actually set the ssl configs which manticore uses for cert verification (:verify).
The option is by default set to strict.

[JasonGiedymin]

CLA Signed.

[JasonGiedymin]

Shall I rebase?

[JasonGiedymin]

Dang, so this repo was possibly finally fixed 19 days ago (actually need to verify) but no comment here. Would have been nice to see at least one heads up from the repo maintainers here so I can either test or update my servers. 👎

[andrewvc]

@JasonGiedymin apologies for the long delay, but this PR slipped through the tracks. Would you mind rebasing? I'll get this merged in ASAP.

[JasonGiedymin]

Will do. I will verify that this "bug" still exists or not.

-Jason

On Jul 19, 2016, at 4:22 PM, Andrew Cholakian [email protected] wrote:

@JasonGiedymin apologies for the long delay, but this PR slipped through the tracks. Would you mind rebasing? I'll get this merged in ASAP.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

[andrewvc]

@JasonGiedymin the current option definitely does not work, but thanks for verifying!

[andrewvc]

@JasonGiedymin actually, it looks like this only applies to hostname verification. Ideally this would work like curl -k. I'm going to research this a bit more.

[andrewvc]

I don't know if this is actually possible with manticore as-is. I've opened the following ticket cheald/manticore#53

[andrewvc]

Closing this given that this is not currently possible to implement with manticore. This option was deprecated in #25

Thanks for the work @JasonGiedymin . If you have the interest in patching manticore we'd love to take a new patch!