Skip to content

Add TLS Check to System Info #7409

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.

Sign up for GitHub

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

Jump to bottom
Open
spencerfinnell opened this issue Oct 4, 2019 · 1 comment · May be fixed by #7439
Open

Add TLS Check to System Info #7409

spencerfinnell opened this issue Oct 4, 2019 · 1 comment · May be fixed by #7439
Labels
type-improvement

Comments

@spencerfinnell
Copy link
Contributor

Enhancement Request

Explain your enhancement (please be detailed)

The System Report currently reports if cURL is enabled, but it would also be nice to show if TLS is enabled as well.

Justification or use case

Many payment gateways (Stripe especially) require TLS to be enabled.

Originally brought up in https://github.com/easydigitaldownloads/edd-stripe/issues/318 but after discussion there it was determined having the information available for all extensions would be the most beneficial.
@cklosowski
Copy link
Contributor

This should be a simple enough thing to achieve.

We could look at using something like this solution on SO: https://stackoverflow.com/a/31351605

It not only tells us if TLS is enabled, but what version it's using, which can be important as well.

@SimonChaw SimonChaw linked a pull request Nov 1, 2019 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type-improvement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added TLS info to system report using cURL SimonChaw/easy-digital-downloads
2 participants
@spencerfinnell @cklosowski