Skip to content
This repository has been archived by the owner on Mar 26, 2021. It is now read-only.

allow insecure certs #38

Merged
merged 4 commits into from
Oct 11, 2018
Merged

allow insecure certs #38

merged 4 commits into from
Oct 11, 2018

Conversation

jgroom33
Copy link
Collaborator

@jgroom33 jgroom33 commented Oct 2, 2018

#37

@jgroom33
Copy link
Collaborator Author

jgroom33 commented Oct 3, 2018

The implementation of the allowInsecure param seemed to be a global setting once it was set for the first request, so I moved it to the base.
It might be better if this was implemented on a per request basis, but that could also be overly explicit.

@eykrehbein eykrehbein merged commit 69dcb97 into eykrehbein:master Oct 11, 2018
eykrehbein added a commit that referenced this pull request Oct 11, 2018
@eykrehbein
Merge pull request #44 from eykrehbein/production
5f471c9 
Version 1.6.0: Allow insecure certs from #38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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.

None yet
2 participants
@jgroom33 @eykrehbein