Skip to content
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

feat: Allows for a user to utilize self-signed certificates #101

Merged
merged 3 commits into from
Oct 19, 2020

Conversation

davcpas1234
Copy link
Contributor

@davcpas1234 davcpas1234 commented Aug 18, 2018

New Feature

Supports #90

Checklist

  • I have made a material change to the repo (functionality, testing, spelling, grammar)
  • I have read the [Contribution Guide] and my PR follows them.
  • I updated my branch with the master branch.
  • I have added in line documentation to the code I modified

Short description of what this PR does:

  • Allows the ability for a user to utilise self-signed certificates with SendGrid
  • Current Client implementation will fail to process a curl_init() function if, for example, the user is testing the implementation on a local server with an unknown CA

If you have questions, please send an email to Sendgrid, or file a Github Issue in this repository.

@thinkingserious thinkingserious added the status: code review request requesting a community code review or review from Twilio label Aug 18, 2018
@SendGridDX
Copy link

SendGridDX commented Aug 18, 2018

CLA assistant check
All committers have signed the CLA.

@codecov
Copy link

codecov bot commented Aug 18, 2018

Codecov Report

Merging #101 into master will increase coverage by 0.43%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #101      +/-   ##
==========================================
+ Coverage    84.7%   85.14%   +0.43%     
==========================================
  Files           3        3              
  Lines         170      175       +5     
==========================================
+ Hits          144      149       +5     
  Misses         26       26
Impacted Files Coverage Δ
lib/Client.php 82.19% <100%> (+0.63%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e9a04d9...cefaee7. Read the comment docs.

davcpas1234 added a commit to davcpas1234/sendgrid-php that referenced this pull request Aug 18, 2018
@thinkingserious thinkingserious added type: community enhancement feature request not on Twilio's roadmap difficulty: medium fix is medium in difficulty labels Aug 21, 2018
@thinkingserious
Copy link
Contributor

Thanks @davcpas1234! This one is our backlog for a code review.

*
* @return Client
*/
public function setVerifySSLCerts($verifySSLCerts)
Copy link
Contributor

Choose a reason for hiding this comment

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

It only has the setVerifySSLCerts method.

Do we need to add the getVerifySSLCerts method?

I think we have this method to get current $verifySSLCerts value.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think it's fine as-is. We don't have getters for the other options.

@childish-sambino childish-sambino removed the type: community enhancement feature request not on Twilio's roadmap label Jul 20, 2020
@thinkingserious thinkingserious changed the base branch from master to main July 28, 2020 14:38
@childish-sambino childish-sambino changed the title Update to Support Self-Signed Certificates feat: Allows for a user to utilize self-signed certificates Oct 19, 2020
@childish-sambino childish-sambino merged commit 7a5c95c into sendgrid:main Oct 19, 2020
childish-sambino pushed a commit to davcpas1234/sendgrid-php that referenced this pull request Oct 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
difficulty: medium fix is medium in difficulty status: code review request requesting a community code review or review from Twilio
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants