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

Update #158 fix for request module and strict SSL #430

Merged
merged 1 commit into from Dec 23, 2015
Merged

Update #158 fix for request module and strict SSL #430

merged 1 commit into from Dec 23, 2015

Conversation

jeffomatic
Copy link
Contributor

Commit 48e8be5 added a compatibility fix for users of the request
NPM module and strict SSL cert checking. The request module has
since been updated (see request/request#1615
for more details), and a corresponding change should be made in
nock to ensure compatibility.

Commit 48e8be5 added a compatibility fix for users of the request
NPM module and strict SSL cert checking. The request module has
since been updated (see request/request#1615
for more details), and a corresponding change should be made in
nock to ensure compatibility.
pgte added a commit that referenced this pull request Dec 23, 2015
Update #158 fix for request module and strict SSL
@pgte pgte merged commit c5be999 into nock:master Dec 23, 2015
@pgte
Copy link
Member

pgte commented Dec 23, 2015

@jeffomatic thanks!

Landed on v3.6.0.

@jeffomatic
Copy link
Contributor Author

Thanks for the super-quick review/merge!

On Wed, Dec 23, 2015 at 4:01 AM, Pedro Teixeira notifications@github.com
wrote:

@jeffomatic https://github.com/jeffomatic thanks!

Landed on v3.6.0.


Reply to this email directly or view it on GitHub
#430 (comment).

@lock
Copy link

lock bot commented Sep 14, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue and add a reference to this one if it’s related. Thank you!

@lock lock bot locked as resolved and limited conversation to collaborators Sep 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants