chore: remove permission sync job

* make updated_by on persmission optional * remove casbin rule model
flanksource · Sep 24, 2024 · 0d520db · 0d520db
1 parent 7d875bf
commit 0d520db
Show file tree

Hide file tree

Showing 4 changed files with 32 additions and 64 deletions.
diff --git a/job/permission.go b/job/permission.go
diff --git a/models/permission.go b/models/permission.go
@@ -8,36 +8,38 @@ import (
 	"github.com/google/uuid"
 )
 
-type CasbinRule struct {
-	ID    int64  `gorm:"primaryKey;autoIncrement"`
-	PType string `json:"ptype"`
-	V0    string `json:"v0"`
-	V1    string `json:"v1"`
-	V2    string `json:"v2"`
-	V3    string `json:"v3"`
-	V4    string `json:"v4"`
-	V5    string `json:"v5"`
-}
-
 type Permission struct {
-	ID          uuid.UUID  `json:"id" gorm:"default:generate_ulid()"`
-	Action      string     `json:"action"`
-	CanaryID    *uuid.UUID `json:"canary_id,omitempty"`
-	ComponentID *uuid.UUID `json:"component_id,omitempty"`
-	ConfigID    *uuid.UUID `json:"config_id,omitempty"`
-	CreatedAt   time.Time  `json:"created_at"`
-	CreatedBy   uuid.UUID  `json:"created_by"`
-	Deny        bool       `json:"deny"`
-	Description string     `json:"description"`
-	PersonID    *uuid.UUID `json:"person_id,omitempty"`
-	PlaybookID  *uuid.UUID `json:"playbook_id,omitempty"`
-	TeamID      *uuid.UUID `json:"team_id,omitempty"`
-	Until       *time.Time `json:"until"`
-	UpdatedAt   *time.Time `json:"updated_at"`
-	UpdatedBy   *uuid.UUID `json:"updated_by"`
+	ID           uuid.UUID  `json:"id" gorm:"default:generate_ulid()"`
+	Action       string     `json:"action"`
+	ConnectionID *uuid.UUID `json:"connection_id,omitempty"`
+	CanaryID     *uuid.UUID `json:"canary_id,omitempty"`
+	ComponentID  *uuid.UUID `json:"component_id,omitempty"`
+	ConfigID     *uuid.UUID `json:"config_id,omitempty"`
+	CreatedAt    time.Time  `json:"created_at"`
+	CreatedBy    uuid.UUID  `json:"created_by"`
+	Deny         bool       `json:"deny"`
+	Description  string     `json:"description"`
+	PersonID     *uuid.UUID `json:"person_id,omitempty"`
+	PlaybookID   *uuid.UUID `json:"playbook_id,omitempty"`
+	TeamID       *uuid.UUID `json:"team_id,omitempty"`
+	Until        *time.Time `json:"until"`
+	UpdatedAt    *time.Time `json:"updated_at"`
+	UpdatedBy    *uuid.UUID `json:"updated_by"`
 }
 
 func (t *Permission) Principal() string {
+	if t.PersonID != nil {
+		return t.PersonID.String()
+	}
+
+	if t.TeamID != nil {
+		return t.TeamID.String()
+	}
+
+	return ""
+}
+
+func (t *Permission) Condition() string {
 	var rule []string
 
 	if t.PersonID != nil {

diff --git a/models/permission_test.go b/models/permission_test.go
@@ -7,7 +7,7 @@ import (
 	"github.com/samber/lo"
 )
 
-func TestPermission_Principal(t *testing.T) {
+func TestPermission_Condition(t *testing.T) {
 	tests := []struct {
 		name     string
 		perm     Permission
@@ -53,7 +53,7 @@ func TestPermission_Principal(t *testing.T) {
 
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
-			result := tt.perm.Principal()
+			result := tt.perm.Condition()
 			if tt.expected != result {
 				t.Errorf("Expected %s, got %s", tt.expected, result)
 			}

diff --git a/schema/permissions.hcl b/schema/permissions.hcl
@@ -58,8 +58,9 @@ table "permissions" {
     null = true
     type = uuid
   }
+
   column "updated_by" {
-    null = false
+    null = true
     type = uuid
   }