diff --git a/client.go b/client.go
index 10bf82fb..e487c295 100644
--- a/client.go
+++ b/client.go
@@ -84,7 +84,6 @@ func GetEndpoints(ctx context.Context, endpoint string, opts ...Option) ([]*ua.E
 // SelectEndpoint returns the endpoint with the highest security level which matches
 // security policy and security mode. policy and mode can be omitted so that
 // only one of them has to match.
-// todo(fs): should this function return an error?
 func SelectEndpoint(endpoints []*ua.EndpointDescription, policy string, mode ua.MessageSecurityMode) (*ua.EndpointDescription, error) {
 	if len(endpoints) == 0 {
 		return nil, errors.Errorf("no endpoints available")