mirror of
https://github.com/hanzoai/tasks.git
synced 2026-10-04 02:04:03 +00:00
## What changed?
<!-- Describe what has changed in this PR -->
Remove license header from every file. Because it is really hard to
follow in this PR here is the summary:
1. License header is removed from all `.go` and `.proto` files
:fireworks::fireworks:🎆.
2. `LICENSE` file in the root directory has only Temporal and Uber
copyrights.
3. 5 other `LICENSE` files added to the packages which have copyrights
different from Temporal and Uber: Datadog, Xargin, "Mat Ryer, Tyler
Bunnell and contributors".
4. `license_file` flag is removed from all code generation tools.
5. `copyright_file` flag is removed from `go:generate mockgen`
directive.
6. All copyright related targets are removed from `Makefile`.
7. Updated Temporal copyright year to 2025 everywhere.
## Why?
<!-- Tell your future self why have you made these changes -->
I double checked with legal department that it is not needed to have
license header in every file. One file per repo is enough. I put all
copyrights to the root `LICENSE` file and removed header from all other
files. Also updated tools and `Makefile`.
182 lines
6.3 KiB
Go
182 lines
6.3 KiB
Go
package authorization
|
|
|
|
import (
|
|
"context"
|
|
"reflect"
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/require"
|
|
"github.com/stretchr/testify/suite"
|
|
"go.temporal.io/server/common/config"
|
|
"go.uber.org/mock/gomock"
|
|
healthpb "google.golang.org/grpc/health/grpc_health_v1"
|
|
)
|
|
|
|
var (
|
|
claimsNone = Claims{}
|
|
claimsNamespaceAdmin = Claims{
|
|
Namespaces: map[string]Role{
|
|
testNamespace: RoleAdmin,
|
|
},
|
|
}
|
|
claimsNamespaceWriter = Claims{
|
|
Namespaces: map[string]Role{
|
|
testNamespace: RoleWriter,
|
|
},
|
|
}
|
|
claimsNamespaceReader = Claims{
|
|
Namespaces: map[string]Role{
|
|
testNamespace: RoleReader,
|
|
},
|
|
}
|
|
claimsBarAdmin = Claims{
|
|
Namespaces: map[string]Role{
|
|
"bar": RoleAdmin,
|
|
},
|
|
}
|
|
claimsSystemAdmin = Claims{
|
|
System: RoleAdmin,
|
|
}
|
|
claimsSystemWriter = Claims{
|
|
System: RoleWriter,
|
|
}
|
|
claimsSystemReader = Claims{
|
|
System: RoleReader,
|
|
}
|
|
targetNamespaceWriteBar = CallTarget{
|
|
APIName: "/temporal.api.workflowservice.v1.WorkflowService/RespondWorkflowTaskCompleted",
|
|
Namespace: "bar",
|
|
}
|
|
targetNamespaceWriteBAR = CallTarget{
|
|
APIName: "/temporal.api.workflowservice.v1.WorkflowService/RespondWorkflowTaskCompleted",
|
|
Namespace: "BAR",
|
|
}
|
|
targetOperatorNamespaceRead = CallTarget{
|
|
APIName: "/temporal.api.operatorservice.v1.OperatorService/ListSearchAttributes",
|
|
Namespace: testNamespace,
|
|
}
|
|
targetGrpcHealthCheck = CallTarget{
|
|
APIName: healthpb.Health_Check_FullMethodName,
|
|
Namespace: "",
|
|
}
|
|
targetGetSystemInfo = CallTarget{
|
|
APIName: "/temporal.api.workflowservice.v1.WorkflowService/GetSystemInfo",
|
|
Namespace: "",
|
|
}
|
|
targetStartWorkflow = CallTarget{
|
|
APIName: "/temporal.api.workflowservice.v1.WorkflowService/StartWorkflowExecution",
|
|
Namespace: testNamespace,
|
|
}
|
|
targetAdminAPI = CallTarget{
|
|
APIName: "/temporal.server.api.adminservice.v1.AdminService/AddSearchAttributes",
|
|
Namespace: testNamespace,
|
|
}
|
|
)
|
|
|
|
type (
|
|
defaultAuthorizerSuite struct {
|
|
suite.Suite
|
|
*require.Assertions
|
|
|
|
controller *gomock.Controller
|
|
authorizer Authorizer
|
|
}
|
|
)
|
|
|
|
func TestDefaultAuthorizerSuite(t *testing.T) {
|
|
s := new(defaultAuthorizerSuite)
|
|
suite.Run(t, s)
|
|
}
|
|
|
|
func (s *defaultAuthorizerSuite) SetupTest() {
|
|
s.Assertions = require.New(s.T())
|
|
s.controller = gomock.NewController(s.T())
|
|
s.authorizer = NewDefaultAuthorizer()
|
|
}
|
|
|
|
func (s *defaultAuthorizerSuite) TearDownTest() {
|
|
s.controller.Finish()
|
|
}
|
|
|
|
func (s *defaultAuthorizerSuite) TestAuthorize() {
|
|
testCases := []struct {
|
|
Name string
|
|
Claims Claims
|
|
Target CallTarget
|
|
Decision Decision
|
|
}{
|
|
// SystemAdmin is allowed on everything
|
|
{"SystemAdminOnFooBar", claimsSystemAdmin, targetNamespaceWriteBar, DecisionAllow},
|
|
{"SystemAdminOnAdminAPI", claimsSystemAdmin, targetAdminAPI, DecisionAllow},
|
|
{"SystemAdminOnStartWorkflow", claimsSystemAdmin, targetStartWorkflow, DecisionAllow},
|
|
|
|
// SystemWriter is allowed on all read only APIs and non-admin APIs on every namespaces
|
|
{"SystemWriterOnFooBar", claimsSystemWriter, targetNamespaceWriteBar, DecisionAllow},
|
|
{"SystemWriterOnAdminAPI", claimsSystemWriter, targetAdminAPI, DecisionDeny},
|
|
{"SystemWriterOnStartWorkflow", claimsSystemWriter, targetStartWorkflow, DecisionAllow},
|
|
|
|
// SystemReader is allowed on all read only APIs and blocked
|
|
{"SystemReaderOnFooBar", claimsSystemReader, targetNamespaceWriteBar, DecisionDeny},
|
|
{"SystemReaderOnAdminAPI", claimsSystemReader, targetAdminAPI, DecisionDeny},
|
|
{"SystemReaderOnStartWorkflow", claimsSystemReader, targetStartWorkflow, DecisionDeny},
|
|
|
|
// NamespaceAdmin is allowed on admin service to their own namespaces (test-namespace)
|
|
{"NamespaceAdminOnAdminAPI", claimsNamespaceAdmin, targetAdminAPI, DecisionDeny},
|
|
{"NamespaceAdminOnStartWorkflow", claimsNamespaceAdmin, targetStartWorkflow, DecisionAllow},
|
|
{"NamespaceAdminOnFooBar", claimsNamespaceAdmin, targetNamespaceWriteBar, DecisionDeny}, // namespace mismatch
|
|
|
|
{"BarAdminOnFooBar", claimsBarAdmin, targetNamespaceWriteBar, DecisionAllow},
|
|
{"BarAdminOnFooBAR", claimsBarAdmin, targetNamespaceWriteBAR, DecisionDeny}, // namespace case mismatch
|
|
|
|
// NamespaceWriter is not allowed on admin APIs
|
|
{"NamespaceWriterOnAdminAPI", claimsNamespaceWriter, targetAdminAPI, DecisionDeny},
|
|
{"NamespaceWriterOnStartWorkflow", claimsNamespaceWriter, targetStartWorkflow, DecisionAllow},
|
|
{"NamespaceWriterOnOperatorNamespaceRead", claimsNamespaceWriter, targetOperatorNamespaceRead, DecisionAllow},
|
|
{"NamespaceWriterOnFooBar", claimsNamespaceWriter, targetNamespaceWriteBar, DecisionDeny}, // namespace mismatch
|
|
|
|
// NamespaceReader is allowed on read-only APIs on non admin service
|
|
{"NamespaceReaderOnAdminAPI", claimsNamespaceReader, targetAdminAPI, DecisionDeny},
|
|
{"NamespaceReaderOnStartWorkflow", claimsNamespaceReader, targetStartWorkflow, DecisionDeny},
|
|
{"NamespaceReaderOnFooBar", claimsNamespaceReader, targetNamespaceWriteBar, DecisionDeny}, // namespace mismatch
|
|
{"NamespaceReaderOnListWorkflow", claimsNamespaceReader, targetGetSystemInfo, DecisionAllow},
|
|
{"NamespaceReaderOnOperatorNamespaceRead", claimsNamespaceReader, targetOperatorNamespaceRead, DecisionAllow},
|
|
|
|
// healthcheck allowed to everyone
|
|
{"RoleNoneOnGetSystemInfo", claimsNone, targetGetSystemInfo, DecisionAllow},
|
|
{"NamespaceReaderOnGetSystemInfo", claimsNamespaceReader, targetGetSystemInfo, DecisionAllow},
|
|
{"RoleNoneOnHealthCheck", claimsNone, targetGrpcHealthCheck, DecisionAllow},
|
|
{"NamespaceReaderOnHealthCheck", claimsNamespaceReader, targetGrpcHealthCheck, DecisionAllow},
|
|
}
|
|
|
|
for _, tt := range testCases {
|
|
result, err := s.authorizer.Authorize(context.TODO(), &tt.Claims, &tt.Target)
|
|
s.NoError(err)
|
|
s.Equal(tt.Decision, result.Decision, "Failed case: %v", tt.Name)
|
|
}
|
|
}
|
|
|
|
func (s *defaultAuthorizerSuite) TestGetAuthorizerFromConfigNoop() {
|
|
s.testGetAuthorizerFromConfig("", true, reflect.TypeOf(&noopAuthorizer{}))
|
|
}
|
|
func (s *defaultAuthorizerSuite) TestGetAuthorizerFromConfigDefault() {
|
|
s.testGetAuthorizerFromConfig("default", true, reflect.TypeOf(&defaultAuthorizer{}))
|
|
}
|
|
func (s *defaultAuthorizerSuite) TestGetAuthorizerFromConfigUnknown() {
|
|
s.testGetAuthorizerFromConfig("foo", false, nil)
|
|
}
|
|
|
|
func (s *defaultAuthorizerSuite) testGetAuthorizerFromConfig(name string, valid bool, authorizerType reflect.Type) {
|
|
|
|
cfg := config.Authorization{Authorizer: name}
|
|
auth, err := GetAuthorizerFromConfig(&cfg)
|
|
if valid {
|
|
s.NoError(err)
|
|
s.NotNil(auth)
|
|
t := reflect.TypeOf(auth)
|
|
s.True(t == authorizerType)
|
|
} else {
|
|
s.Error(err)
|
|
s.Nil(auth)
|
|
}
|
|
}
|