feat: servicedata

Makefile

@@ -2,7 +2,7 @@ GOVERSION := $(shell go version | cut -d ' ' -f 3 | cut -d '.' -f 2)
 
 .PHONY: build check fmt lint test test-race vet test-cover-html help install proto
 .DEFAULT_GOAL := build
-PROTON_COMMIT := "e8a584e192f23f999844b027d17bd738c3981973"
+PROTON_COMMIT := "6ee59f2d0cbeedf1d5fe48adee5e5e41f54f081e"
 
 install:
 	@echo "Clean up imports..."

core/group/filter.go

@@ -1,5 +1,7 @@
 package group
 
 type Filter struct {
-	OrganizationID string
+	OrganizationID            string
+	ProjectID                 string
+	ServicedataKeyResourceIDs []string
 }

core/servicedata/service.go

@@ -15,6 +15,7 @@ import (
 	"github.com/goto/shield/core/user"
 	"github.com/goto/shield/internal/schema"
 	"github.com/goto/shield/pkg/db"
+	"github.com/goto/shield/pkg/errors"
 )
 
 const (
@@ -75,7 +76,7 @@ func NewService(logger log.Logger, repository Repository, resourceService Resour
 
 func (s Service) CreateKey(ctx context.Context, key Key) (Key, error) {
 	// check if key contains ':'
-	if key.Key == "" {
+	if key.Name == "" {
 		return Key{}, ErrInvalidDetail
 	}
 
@@ -160,7 +161,7 @@ func (s Service) CreateKey(ctx context.Context, key Key) (Key, error) {
 }
 
 func (s Service) Upsert(ctx context.Context, sd ServiceData) (ServiceData, error) {
-	if sd.Key.Key == "" {
+	if sd.Key.Name == "" {
 		return ServiceData{}, ErrInvalidDetail
 	}
 
@@ -188,7 +189,7 @@ func (s Service) Upsert(ctx context.Context, sd ServiceData) (ServiceData, error
 		return ServiceData{}, err
 	}
 	if !permission {
-		return ServiceData{}, user.ErrInvalidEmail
+		return ServiceData{}, errors.ErrForbidden
 	}
 
 	returnedServiceData, err := s.repository.Upsert(ctx, sd)

core/servicedata/service_test.go

@@ -14,7 +14,9 @@ import (
 	"github.com/goto/shield/core/servicedata/mocks"
 	"github.com/goto/shield/core/user"
 	"github.com/goto/shield/internal/schema"
+	errorsPkg "github.com/goto/shield/pkg/errors"
 	"github.com/goto/shield/pkg/logger"
+
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/mock"
 )
@@ -30,21 +32,21 @@ var (
 	testProjectSlug = "test-project-slug"
 	testKey         = servicedata.Key{
 		ProjectID:   "test-project-slug",
-		Key:         "test-key",
+		Name:        "test-key",
 		Description: "test key no 01",
 	}
 	testCreateKey = servicedata.Key{
 		URN:         "test-project-slug:servicedata_key:test-key",
 		ProjectID:   testProjectID,
 		ProjectSlug: testProjectSlug,
-		Key:         "test-key",
+		Name:        "test-key",
 		Description: "test key no 01",
 		ResourceID:  testResourceID,
 	}
 	testCreatedKey = servicedata.Key{
 		URN:         "test-project-slug:servicedata_key:test-key",
 		ProjectID:   testProjectID,
-		Key:         "test-key",
+		Name:        "test-key",
 		Description: "test key no 01",
 		ResourceID:  testResourceID,
 	}
@@ -131,7 +133,7 @@ func TestService_CreateKey(t *testing.T) {
 			name: "CreateKeyEmpty",
 			key: servicedata.Key{
 				ProjectID:   testKey.ProjectID,
-				Key:         "",
+				Name:        "",
 				Description: testKey.Description,
 			},
 			setup: func(t *testing.T) *servicedata.Service {
@@ -183,7 +185,7 @@ func TestService_CreateKey(t *testing.T) {
 			name: "CreateKeyInvalidProjectID",
 			key: servicedata.Key{
 				ProjectID:   "invalid-test-project-slug",
-				Key:         testKey.Key,
+				Name:        testKey.Name,
 				Description: testKey.Description,
 			},
 			email: "[email protected]",
@@ -366,7 +368,7 @@ func TestService_Upsert(t *testing.T) {
 			name: "UpsertKeyEmpty",
 			data: servicedata.ServiceData{
 				Key: servicedata.Key{
-					Key: "",
+					Name: "",
 				},
 			},
 			setup: func(t *testing.T) *servicedata.Service {
@@ -402,7 +404,7 @@ func TestService_Upsert(t *testing.T) {
 			name: "UpsertInvalidProjectID",
 			data: servicedata.ServiceData{
 				Key: servicedata.Key{
-					Key:       testKey.Key,
+					Name:      testKey.Name,
 					ProjectID: "invalid-test-project-slug",
 				},
 			},
@@ -480,7 +482,7 @@ func TestService_Upsert(t *testing.T) {
 					testResourceID, action.Action{ID: "edit"}).Return(false, nil)
 				return servicedata.NewService(testLogger, repository, resourceService, relationService, projectService, userService, activityService)
 			},
-			wantErr: user.ErrInvalidEmail,
+			wantErr: errorsPkg.ErrForbidden,
 		},
 		{
 			name:  "UpsertErr",

core/servicedata/servicedata.go

@@ -26,7 +26,7 @@ type Key struct {
 	URN         string
 	ProjectID   string
 	ProjectSlug string
-	Key         string
+	Name        string
 	Description string
 	ResourceID  string
 }
@@ -36,7 +36,7 @@ type ServiceData struct {
 	NamespaceID string
 	EntityID    string
 	Key         Key
-	Value       string
+	Value       any
 }
 
 type KeyLogData struct {
@@ -56,15 +56,15 @@ type Filter struct {
 }
 
 func (key Key) CreateURN() string {
-	return fmt.Sprintf("%s:servicedata_key:%s", key.ProjectSlug, key.Key)
+	return fmt.Sprintf("%s:servicedata_key:%s", key.ProjectSlug, key.Name)
 }
 
 func (key Key) ToKeyLogData() KeyLogData {
 	return KeyLogData{
 		Entity:      auditEntityServiceDataKey,
 		URN:         key.URN,
 		ProjectSlug: key.ProjectSlug,
-		Key:         key.Key,
+		Key:         key.Name,
 		Description: key.Description,
 	}
 }

core/user/filter.go

@@ -1,7 +1,9 @@
 package user
 
 type Filter struct {
-	Limit   int32
-	Page    int32
-	Keyword string
+	Limit                     int32
+	Page                      int32
+	Keyword                   string
+	ProjectID                 string
+	ServiceDataKeyResourceIds []string
 }