From 51525ea2ba744310af019fdbbf74aa23d89cb56d Mon Sep 17 00:00:00 2001 From: chrislu Date: Mon, 25 Aug 2025 23:29:18 -0700 Subject: [PATCH] remove filerAddress required --- weed/iam/integration/role_store.go | 9 +-------- weed/iam/policy/policy_store.go | 9 +-------- 2 files changed, 2 insertions(+), 16 deletions(-) diff --git a/weed/iam/integration/role_store.go b/weed/iam/integration/role_store.go index c34c1a2b2..295115be4 100644 --- a/weed/iam/integration/role_store.go +++ b/weed/iam/integration/role_store.go @@ -146,18 +146,11 @@ func NewFilerRoleStore(config map[string]interface{}) (*FilerRoleStore, error) { basePath: "/etc/iam/roles", // Default path for role storage - aligned with /etc/ convention } - // Parse configuration + // Parse configuration - only basePath and other settings, NOT filerAddress if config != nil { if basePath, ok := config["basePath"].(string); ok && basePath != "" { store.basePath = strings.TrimSuffix(basePath, "/") } - - // Validate that filerAddress is provided in config - required for distributed setup - if _, ok := config["filerAddress"].(string); !ok { - return nil, fmt.Errorf("filer address is required in configuration for FilerRoleStore") - } - } else { - return nil, fmt.Errorf("filer address is required in configuration for FilerRoleStore") } glog.V(2).Infof("Initialized FilerRoleStore with basePath %s", store.basePath) diff --git a/weed/iam/policy/policy_store.go b/weed/iam/policy/policy_store.go index c44afd6e4..4c673d40a 100644 --- a/weed/iam/policy/policy_store.go +++ b/weed/iam/policy/policy_store.go @@ -158,18 +158,11 @@ func NewFilerPolicyStore(config map[string]interface{}) (*FilerPolicyStore, erro basePath: "/etc/iam/policies", // Default path for policy storage - aligned with /etc/ convention } - // Parse configuration + // Parse configuration - only basePath and other settings, NOT filerAddress if config != nil { if basePath, ok := config["basePath"].(string); ok && basePath != "" { store.basePath = strings.TrimSuffix(basePath, "/") } - - // Validate that filerAddress is provided in config - required for distributed setup - if _, ok := config["filerAddress"].(string); !ok { - return nil, fmt.Errorf("filer address is required in configuration for FilerPolicyStore") - } - } else { - return nil, fmt.Errorf("filer address is required in configuration for FilerPolicyStore") } glog.V(2).Infof("Initialized FilerPolicyStore with basePath %s", store.basePath)