-
Notifications
You must be signed in to change notification settings - Fork 6
Conversation
app/analyze/executors/shell.go
Outdated
@@ -67,10 +67,10 @@ func trackMemoryEveryNSeconds(ctx context.Context, name string, pid int) { | |||
} | |||
} | |||
|
|||
func (s shell) wait(ctx context.Context, name string, pid int, outReader io.ReadCloser) []string { | |||
func (s shell) wait(ctx context.Context, name string, childPid int, outReader io.ReadCloser) []string { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
outReader can be io.Reader
app/analyze/state/api_storage.go
Outdated
@@ -28,7 +28,9 @@ func getHTTPClient() *http.Client { | |||
} | |||
|
|||
tr := &http.Transport{ | |||
TLSClientConfig: &tls.Config{InsecureSkipVerify: true}, | |||
TLSClientConfig: &tls.Config{ | |||
InsecureSkipVerify: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
G402: TLS InsecureSkipVerify set true.
c298973
to
488ad3c
Compare
app/analytics/track.go
Outdated
UserId: userID, | ||
EventType: string(eventName), | ||
EventProperties: eventProps, | ||
}) | ||
} | ||
ac.Publish(ev) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Error return value of ac.Publish
is not checked
app/analyze/state/api_storage.go
Outdated
@@ -28,7 +28,9 @@ func getHTTPClient() *http.Client { | |||
} | |||
|
|||
tr := &http.Transport{ | |||
TLSClientConfig: &tls.Config{InsecureSkipVerify: true}, | |||
TLSClientConfig: &tls.Config{ | |||
InsecureSkipVerify: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
G402: TLS InsecureSkipVerify set true.
app/analyze/executors/shell.go
Outdated
@@ -67,10 +67,10 @@ func trackMemoryEveryNSeconds(ctx context.Context, name string, pid int) { | |||
} | |||
} | |||
|
|||
func (s shell) wait(ctx context.Context, name string, pid int, outReader io.ReadCloser) []string { | |||
func (s shell) wait(ctx context.Context, name string, childPid int, outReader io.ReadCloser) []string { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
outReader can be io.Reader
app/analyze/executors/shell.go
Outdated
@@ -67,10 +67,10 @@ func trackMemoryEveryNSeconds(ctx context.Context, name string, pid int) { | |||
} | |||
} | |||
|
|||
func (s shell) wait(ctx context.Context, name string, pid int, outReader io.ReadCloser) []string { | |||
func (s shell) wait(ctx context.Context, name string, childPid int, outReader io.ReadCloser) []string { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
outReader can be io.Reader
app/analyze/state/api_storage.go
Outdated
@@ -28,7 +28,9 @@ func getHTTPClient() *http.Client { | |||
} | |||
|
|||
tr := &http.Transport{ | |||
TLSClientConfig: &tls.Config{InsecureSkipVerify: true}, | |||
TLSClientConfig: &tls.Config{ | |||
InsecureSkipVerify: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
G402: TLS InsecureSkipVerify set true.
app/analytics/track.go
Outdated
UserId: userID, | ||
EventType: string(eventName), | ||
EventProperties: eventProps, | ||
}) | ||
} | ||
ac.Publish(ev) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Error return value of ac.Publish
is not checked
app/analytics/track.go
Outdated
UserId: userID, | ||
EventType: string(eventName), | ||
EventProperties: eventProps, | ||
}) | ||
} | ||
ac.Publish(ev) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Error return value of ac.Publish
is not checked
app/analyze/state/api_storage.go
Outdated
@@ -28,7 +28,9 @@ func getHTTPClient() *http.Client { | |||
} | |||
|
|||
tr := &http.Transport{ | |||
TLSClientConfig: &tls.Config{InsecureSkipVerify: true}, | |||
TLSClientConfig: &tls.Config{ | |||
InsecureSkipVerify: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
G402: TLS InsecureSkipVerify set true.
app/analyze/executors/shell.go
Outdated
@@ -67,10 +67,10 @@ func trackMemoryEveryNSeconds(ctx context.Context, name string, pid int) { | |||
} | |||
} | |||
|
|||
func (s shell) wait(ctx context.Context, name string, pid int, outReader io.ReadCloser) []string { | |||
func (s shell) wait(ctx context.Context, name string, childPid int, outReader io.ReadCloser) []string { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
outReader can be io.Reader
488ad3c
to
c6d88cd
Compare
c6d88cd
to
5d119ec
Compare
No description provided.