Make getProviderAccessToken private (#1506)

Signed-off-by: Radoslav Dimitrov <radoslav@stacklok.com>
mindersec · Nov 6, 2023 · 9598c40 · 9598c40
1 parent 6712ee4
commit 9598c40
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/internal/controlplane/handlers_oauth.go b/internal/controlplane/handlers_oauth.go
@@ -257,8 +257,8 @@ func (s *Server) ExchangeCodeForTokenWEB(ctx context.Context,
 	}, nil
 }
 
-// GetProviderAccessToken returns the access token for providers
-func (s *Server) GetProviderAccessToken(ctx context.Context, provider string,
+// getProviderAccessToken returns the access token for providers
+func (s *Server) getProviderAccessToken(ctx context.Context, provider string,
 	projectID uuid.UUID, checkAuthz bool) (oauth2.Token, string, error) {
 	// check if user is authorized
 	if checkAuthz {

diff --git a/internal/controlplane/handlers_repositories.go b/internal/controlplane/handlers_repositories.go
@@ -427,7 +427,7 @@ func (s *Server) ListRemoteRepositoriesFromProvider(
 	}
 
 	// FIXME: this is a hack to get the owner filter from the request
-	_, owner_filter, err := s.GetProviderAccessToken(ctx, provider.Name, projectID, true)
+	_, owner_filter, err := s.getProviderAccessToken(ctx, provider.Name, projectID, true)
 
 	if err != nil {
 		return nil, status.Errorf(codes.PermissionDenied, "cannot get access token for provider")