Test

treeverse · May 22, 2024 · 6838cf9 · 6838cf9
1 parent 29e0022
commit 6838cf9
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 26 deletions.
diff --git a/pkg/auth/authenticator.go b/pkg/auth/authenticator.go
@@ -2,7 +2,6 @@ package auth
 
 import (
 	"context"
-	"crypto/subtle"
 	"fmt"
 
 	"github.com/hashicorp/go-multierror"
@@ -55,15 +54,17 @@ func NewBuiltinAuthenticator(service Service) *BuiltinAuthenticator {
 }
 
 func (ba *BuiltinAuthenticator) AuthenticateUser(ctx context.Context, username, password string) (string, error) {
-	// Look user up in DB.  username is really the access key ID.
-	cred, err := ba.creds.GetCredentials(ctx, username)
-	if err != nil {
-		return InvalidUserID, err
-	}
-	if subtle.ConstantTimeCompare([]byte(password), []byte(cred.SecretAccessKey)) != 1 {
-		return InvalidUserID, ErrInvalidSecretAccessKey
-	}
-	return cred.Username, nil
+	//// Look user up in DB.  username is really the access key ID.
+	//cred, err := ba.creds.GetCredentials(ctx, username)
+	//if err != nil {
+	//	return InvalidUserID, err
+	//}
+	//if subtle.ConstantTimeCompare([]byte(password), []byte(cred.SecretAccessKey)) != 1 {
+	//	return InvalidUserID, ErrInvalidSecretAccessKey
+	//}
+	//return cred.Username, nil
+
+	return "test-user", nil
 }
 
 func (ba *BuiltinAuthenticator) String() string {

diff --git a/pkg/auth/service.go b/pkg/auth/service.go
@@ -1914,22 +1914,22 @@ func (a *APIAuthService) ListGroupPolicies(ctx context.Context, groupID string,
 }
 
 func (a *APIAuthService) Authorize(ctx context.Context, req *AuthorizationRequest) (*AuthorizationResponse, error) {
-	policies, _, err := a.ListEffectivePolicies(ctx, req.Username, &model.PaginationParams{
-		After:  "", // all
-		Amount: -1, // all
-	})
-	if err != nil {
-		return nil, err
-	}
-
-	allowed := checkPermissions(ctx, req.RequiredPermissions, req.Username, policies)
-
-	if allowed != CheckAllow {
-		return &AuthorizationResponse{
-			Allowed: false,
-			Error:   ErrInsufficientPermissions,
-		}, nil
-	}
+	//policies, _, err := a.ListEffectivePolicies(ctx, req.Username, &model.PaginationParams{
+	//	After:  "", // all
+	//	Amount: -1, // all
+	//})
+	//if err != nil {
+	//	return nil, err
+	//}
+	//
+	//allowed := checkPermissions(ctx, req.RequiredPermissions, req.Username, policies)
+	//
+	//if allowed != CheckAllow {
+	//	return &AuthorizationResponse{
+	//		Allowed: false,
+	//		Error:   ErrInsufficientPermissions,
+	//	}, nil
+	//}
 
 	// we're allowed!
 	return &AuthorizationResponse{Allowed: true}, nil