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.

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

Consider strict-ssl npm config #100

Closed
wants to merge 1 commit into from
Closed

Consider strict-ssl npm config #100

wants to merge 1 commit into from

Conversation

shinnn
Copy link
Collaborator

@shinnn shinnn commented May 12, 2016 •
edited

fix #97
fix imagemin/optipng-bin#74
close #101

If strict-ssl npm config is disabled, this module doesn't emit errors even if the server certificate is not verified.

@kevva, otherwise, should this feature is included in https://github.com/kevva/bin-build?

@kevva
Copy link
Owner

kevva commented May 21, 2016

@shinnn, can you rebase this? I'll merge it :).

@shinnn
Copy link
Collaborator Author

shinnn commented May 21, 2016

@kevva Sure.

@kevva kevva closed this in 559a6f5 Jun 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

enable non strict-ssl install option Unable to verify the first certificate
2 participants
@shinnn @kevva