fix: Fixed GET function in ZIA urlfilteringpolicies (#408)

* fix: Fixed  function in ZIA

Author: willguibr

CHANGELOG.md

@@ -1,5 +1,14 @@
 # Changelog
 
+# 3.8.18 (February 10, 2026)
+
+## Notes
+- Golang: **v1.24**
+
+### Bug Fixes
+
+- [PR #408](https://github.com/zscaler/zscaler-sdk-go/pull/408) - Fixed `GET` function in ZIA `urlfilteringpolicies`.
+
 # 3.8.17 (February 9, 2026)
 
 ## Notes

docs/guides/release-notes.md

@@ -13,10 +13,19 @@ Track all Zscaler SDK GO releases. New resources, features, and bug fixes will b
 
 ---
 
-``Last updated: v3.8.17``
+``Last updated: v3.8.18``
 
 ---
 
+# 3.8.18 (February 10, 2026)
+
+## Notes
+- Golang: **v1.24**
+
+### Bug Fixes
+
+- [PR #408](https://github.com/zscaler/zscaler-sdk-go/pull/408) - Fixed `GET` function in ZIA `urlfilteringpolicies`.
+
 # 3.8.17 (February 9, 2026)
 
 ## Notes

go.mod

@@ -17,7 +17,7 @@ require (
 	github.com/olekukonko/tablewriter v0.0.5
 	github.com/stretchr/testify v1.11.1
 	github.com/zscaler/zscaler-sdk-go/v2 v2.732.0
-	golang.org/x/text v0.33.0
+	golang.org/x/text v0.34.0
 	gopkg.in/dnaeon/go-vcr.v4 v4.0.6
 	gopkg.in/yaml.v3 v3.0.1
 )

go.sum

@@ -480,8 +480,8 @@ golang.org/x/text v0.3.7/go.mod h1:u+2+/6zg+i71rQMx5EYifcz6MCKuco9NR6JIITiCfzQ=
 golang.org/x/text v0.7.0/go.mod h1:mrYo+phRRbMaCq/xk9113O4dZlRixOauAjOtrjsXDZ8=
 golang.org/x/text v0.9.0/go.mod h1:e1OnstbJyHTd6l/uOt8jFFHp6TRDWZR/bV3emEE/zU8=
 golang.org/x/text v0.14.0/go.mod h1:18ZOQIKpY8NJVqYksKHtTdi31H5itFRjB5/qKTNYzSU=
-golang.org/x/text v0.33.0 h1:B3njUFyqtHDUI5jMn1YIr5B0IE2U0qck04r6d4KPAxE=
-golang.org/x/text v0.33.0/go.mod h1:LuMebE6+rBincTi9+xWTY8TztLzKHc/9C1uBCG27+q8=
+golang.org/x/text v0.34.0 h1:oL/Qq0Kdaqxa1KbNeMKwQq0reLCCaFtqu2eNuSeNHbk=
+golang.org/x/text v0.34.0/go.mod h1:homfLqTYRFyVYemLBFl5GgL/DWEiH5wcsQ5gSh1yziA=
 golang.org/x/time v0.0.0-20181108054448-85acf8d2951c/go.mod h1:tRJNPiyCQ0inRvYxbN9jk5I+vvW/OXSQhTDSoE431IQ=
 golang.org/x/time v0.0.0-20190308202827-9d24e82272b4/go.mod h1:tRJNPiyCQ0inRvYxbN9jk5I+vvW/OXSQhTDSoE431IQ=
 golang.org/x/time v0.0.0-20191024005414-555d28b269f0/go.mod h1:tRJNPiyCQ0inRvYxbN9jk5I+vvW/OXSQhTDSoE431IQ=

tests/unit/zia/services/urlfiltering_test.go

@@ -21,12 +21,11 @@ func TestURLFilteringRules_Get_SDK(t *testing.T) {
 	defer server.Close()
 
 	ruleID := 12345
-	path := "/zia/api/v1/urlFilteringRules"
+	path := "/zia/api/v1/urlFilteringRules/12345"
 
-	// Get uses GetAll internally and finds by ID
-	server.On("GET", path, common.SuccessResponse([]urlfilteringpolicies.URLFilteringRule{
-		{ID: 1, Name: "Other Rule", Action: "ALLOW"},
-		{ID: ruleID, Name: "Block Social Media", Action: "BLOCK", State: "ENABLED"},
+	// Get retrieves a single rule by ID via direct API call
+	server.On("GET", path, common.SuccessResponse(urlfilteringpolicies.URLFilteringRule{
+		ID: ruleID, Name: "Block Social Media", Action: "BLOCK", State: "ENABLED",
 	}))
 
 	service, err := common.CreateTestService(context.Background(), server, "123456")
@@ -115,11 +114,10 @@ func TestURLFilteringRules_Update_SDK(t *testing.T) {
 
 	ruleID := 12345
 	path := "/zia/api/v1/urlFilteringRules/12345"
-	listPath := "/zia/api/v1/urlFilteringRules"
 
-	// Mock GetAll call that Update performs to fetch CBIProfile
-	server.On("GET", listPath, common.SuccessResponse([]urlfilteringpolicies.URLFilteringRule{
-		{ID: ruleID, Name: "Updated URL Rule", CBIProfile: &urlfilteringpolicies.CBIProfile{ID: "cbi-id"}},
+	// Mock Get call that Update performs to fetch CBIProfile (direct by ID)
+	server.On("GET", path, common.SuccessResponse(urlfilteringpolicies.URLFilteringRule{
+		ID: ruleID, Name: "Updated URL Rule", CBIProfile: &urlfilteringpolicies.CBIProfile{ID: "cbi-id"},
 	}))
 
 	server.On("PUT", path, common.SuccessResponse(urlfilteringpolicies.URLFilteringRule{

zscaler/zia/services/urlcategories/urlcategories.go

@@ -63,6 +63,9 @@ type URLCategory struct {
 	// Super Category of the URL category. This field is required when creating custom URL categories.
 	SuperCategory string `json:"superCategory,omitempty"`
 
+	// Super Category of the URL category. This field is required when creating custom URL categories.
+	CategoryGroup string `json:"categoryGroup,omitempty"`
+
 	// Specifies the type of URL match, such as using exact URLs or regex patterns.
 	// For regex, the patterns can be specified using the regexPatterns and regexPatternsRetainingParentCategory fields.
 	// For exact URLs, specify the required URLs, keywords, or IP ranges using the appropriate fields.

zscaler/zia/services/urlfilteringpolicies/urlfilteringpolicies.go

@@ -214,20 +214,14 @@ type URLAdvancedPolicySettings struct {
 }
 
 func Get(ctx context.Context, service *zscaler.Service, ruleID int) (*URLFilteringRule, error) {
-	var urlFilteringPolicies []URLFilteringRule
-	err := common.ReadAllPages(ctx, service.Client, urlFilteringPoliciesEndpoint, &urlFilteringPolicies)
+	var rule URLFilteringRule
+	err := service.Client.Read(ctx, fmt.Sprintf("%s/%d", urlFilteringPoliciesEndpoint, ruleID), &rule)
 	if err != nil {
 		return nil, err
 	}
 
-	for _, urlFilteringPolicy := range urlFilteringPolicies {
-		if urlFilteringPolicy.ID == ruleID {
-			service.Client.GetLogger().Printf("[DEBUG]Returning url filtering rules from Get: %d", urlFilteringPolicy.ID)
-			return &urlFilteringPolicy, nil
-		}
-	}
-
-	return nil, fmt.Errorf("no url filtering rule found with id: %d", ruleID)
+	service.Client.GetLogger().Printf("[DEBUG]Returning url filtering rules from Get: %d", rule.ID)
+	return &rule, nil
 }
 
 func GetByName(ctx context.Context, service *zscaler.Service, urlFilteringPolicyName string) (*URLFilteringRule, error) {
@@ -265,17 +259,11 @@ func Update(ctx context.Context, service *zscaler.Service, ruleID int, rule *URL
 
 	// Check if CBIProfile is nil or has empty ID, or if CBIProfileID is 0
 	if rule.CBIProfile == nil || rule.CBIProfile.ID == "" || rule.CBIProfileID == 0 {
-		// If CBIProfile object is empty, fetch it using GetByName as Get by ID is not currently returnign the full CBIProfile object with the uuid ID
-		var urlFilteringPolicies []URLFilteringRule
-		err := common.ReadAllPages(ctx, service.Client, urlFilteringPoliciesEndpoint, &urlFilteringPolicies)
-		if err != nil {
-			return nil, nil, err
-		}
-		for _, urlFilteringPolicy := range urlFilteringPolicies {
-			if urlFilteringPolicy.ID == ruleID {
-				rule.CBIProfile = urlFilteringPolicy.CBIProfile
-				rule.CBIProfileID = urlFilteringPolicy.CBIProfileID
-			}
+		// If CBIProfile object is empty, fetch it using Get by ID
+		existingRule, err := Get(ctx, service, ruleID)
+		if err == nil && existingRule != nil {
+			rule.CBIProfile = existingRule.CBIProfile
+			rule.CBIProfileID = existingRule.CBIProfileID
 		}
 	}
 	resp, err := service.Client.UpdateWithPut(ctx, fmt.Sprintf("%s/%d", urlFilteringPoliciesEndpoint, ruleID), *rule)
@@ -299,12 +287,12 @@ func Delete(ctx context.Context, service *zscaler.Service, ruleID int) (*http.Re
 
 // GetAll returns the all rules.
 func GetAll(ctx context.Context, service *zscaler.Service) ([]URLFilteringRule, error) {
-	var urlFilteringPolicies []URLFilteringRule
-	err := common.ReadAllPages(ctx, service.Client, urlFilteringPoliciesEndpoint, &urlFilteringPolicies)
-	if err != nil {
-		return nil, err
-	}
-	return urlFilteringPolicies, nil
+	var rules []URLFilteringRule
+
+	// Use service.Client.Read directly since the API doesn't support pagination
+	// The API returns all results in a single response
+	err := service.Client.Read(ctx, urlFilteringPoliciesEndpoint, &rules)
+	return rules, err
 }
 
 func GetUrlAndAppSettings(ctx context.Context, service *zscaler.Service) (*URLAdvancedPolicySettings, error) {