Merge pull request #5 from development

ZachtimusPrime · web-flow · commit 2e2b5d49cd3c · 2019-01-29T14:32:10.000-06:00
Remove insecure transport on default client
diff --git a/splunk/splunk.go b/splunk/splunk.go
@@ -44,7 +44,7 @@ type Client struct {
 func NewClient(httpClient *http.Client, URL string, Token string, Source string, SourceType string, Index string) *Client {
 	// Create a new client
 	if httpClient == nil {
-		tr := &http.Transport{TLSClientConfig: &tls.Config{InsecureSkipVerify: true}} // turn off certificate checking
+		tr := &http.Transport{TLSClientConfig: &tls.Config{InsecureSkipVerify: false}}
 		httpClient = &http.Client{Timeout: time.Second * 20, Transport: tr}
 	}
 	hostname, _ := os.Hostname()

Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,7 @@ type Client struct {`
`44`	`44`	`func NewClient(httpClient http.Client, URL string, Token string, Source string, SourceType string, Index string) Client {`
`45`	`45`	`// Create a new client`
`46`	`46`	`if httpClient == nil {`
`47`		`- tr := &http.Transport{TLSClientConfig: &tls.Config{InsecureSkipVerify: true}} // turn off certificate checking`
	`47`	`+ tr := &http.Transport{TLSClientConfig: &tls.Config{InsecureSkipVerify: false}}`
`48`	`48`	`httpClient = &http.Client{Timeout: time.Second * 20, Transport: tr}`
`49`	`49`	`}`
`50`	`50`	`hostname, _ := os.Hostname()`