parameter 'orgID' seems to be unused, consider removing or renaming it as _
61 return apiClient.RevokeLimitedAccessToken(ctx, cachedToken.ID)
62}
63
64func addBuildTokenCaveats(m *macaroon.Macaroon, orgID uint64, includeExpiry bool) { 65 action := resset.ActionRead | resset.ActionWrite | resset.ActionCreate | resset.ActionDelete
66
67 m.Add(&resset.IfPresent{parameter 'ctx' seems to be unused, consider removing or renaming it as _
85 return nil, status.Errorf(codes.Unavailable, "client side tokens disabled")
86}
87
88func (ap *buildkitAuthProvider) VerifyTokenAuthority(ctx context.Context, req *auth.VerifyTokenAuthorityRequest) (*auth.VerifyTokenAuthorityResponse, error) {89 return nil, status.Errorf(codes.Unavailable, "client side tokens disabled")
90}
parameter 'req' seems to be unused, consider removing or renaming it as _
85 return nil, status.Errorf(codes.Unavailable, "client side tokens disabled")
86}
87
88func (ap *buildkitAuthProvider) VerifyTokenAuthority(ctx context.Context, req *auth.VerifyTokenAuthorityRequest) (*auth.VerifyTokenAuthorityResponse, error) {89 return nil, status.Errorf(codes.Unavailable, "client side tokens disabled")
90}
parameter 'req' seems to be unused, consider removing or renaming it as _
81 return nil, status.Errorf(codes.Unavailable, "client side tokens disabled")
82}
83
84func (ap *buildkitAuthProvider) GetTokenAuthority(ctx context.Context, req *auth.GetTokenAuthorityRequest) (*auth.GetTokenAuthorityResponse, error) {85 return nil, status.Errorf(codes.Unavailable, "client side tokens disabled")
86}
87parameter 'ctx' seems to be unused, consider removing or renaming it as _
81 return nil, status.Errorf(codes.Unavailable, "client side tokens disabled")
82}
83
84func (ap *buildkitAuthProvider) GetTokenAuthority(ctx context.Context, req *auth.GetTokenAuthorityRequest) (*auth.GetTokenAuthorityResponse, error) {85 return nil, status.Errorf(codes.Unavailable, "client side tokens disabled")
86}
87parameter 'ctx' seems to be unused, consider removing or renaming it as _
77 return res, nil
78}
79
80func (ap *buildkitAuthProvider) FetchToken(ctx context.Context, req *auth.FetchTokenRequest) (*auth.FetchTokenResponse, error) {81 return nil, status.Errorf(codes.Unavailable, "client side tokens disabled")
82}
83parameter 'req' seems to be unused, consider removing or renaming it as _
77 return res, nil
78}
79
80func (ap *buildkitAuthProvider) FetchToken(ctx context.Context, req *auth.FetchTokenRequest) (*auth.FetchTokenResponse, error) {81 return nil, status.Errorf(codes.Unavailable, "client side tokens disabled")
82}
83
Description
Unused parameters in functions or methods should be replaced with _
(underscore) or removed.
Functions or methods with unused parameters can be a symptom of unfinished
refactoring or a bug. If an unused parameter is present, it should be named _
(underscore) to avoid raising this issue and better readability.
Bad practice
func abc(unused string) {
fmt.Println("Not using any passed params.")
}
Recommended
func abc(_ string) {
fmt.Println("Not using any passed params.")
}
func abc() {
fmt.Println("Not using any passed params.")
}