Skip to content

Commit

Permalink
Fix linter error with updated linter
Browse files Browse the repository at this point in the history
Signed-off-by: Phil Estes <estesp@amazon.com>
  • Loading branch information
estesp committed Dec 1, 2023
1 parent b638791 commit a1ae572
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 5 deletions.
3 changes: 1 addition & 2 deletions integration/util/util_windows.go
Original file line number Diff line number Diff line change
Expand Up @@ -117,9 +117,8 @@ func parseEndpoint(endpoint string) (string, string, error) {
return "npipe", fmt.Sprintf("//%s%s", host, u.Path), nil
} else if u.Scheme == "" {
return "", "", fmt.Errorf("Using %q as endpoint is deprecated, please consider using full url format", endpoint)
} else {
return u.Scheme, "", fmt.Errorf("protocol %q not supported", u.Scheme)
}
return u.Scheme, "", fmt.Errorf("protocol %q not supported", u.Scheme)
}

// LocalEndpoint empty implementation
Expand Down
5 changes: 2 additions & 3 deletions tracing/plugin/otlp.go
Original file line number Diff line number Diff line change
Expand Up @@ -117,10 +117,9 @@ func newExporter(ctx context.Context, cfg *OTLPConfig) (*otlptrace.Exporter, err
opts = append(opts, otlptracegrpc.WithInsecure())
}
return otlptracegrpc.New(ctx, opts...)
} else {
// Other protocols such as "http/json" are not supported.
return nil, fmt.Errorf("OpenTelemetry protocol %q is not supported", cfg.Protocol)
}
// Other protocols such as "http/json" are not supported.
return nil, fmt.Errorf("OpenTelemetry protocol %q is not supported", cfg.Protocol)
}

// newTracer configures protocol-agonostic tracing settings such as
Expand Down

0 comments on commit a1ae572

Please sign in to comment.