feat: remove required parameters from user identifier policy conditions (#589)

Author: pranav-okta

.generator/okta-management-APIs-oasv3-noEnums-inheritance.yaml

@@ -54180,9 +54180,6 @@ components:
         value:
           type: string
           description: The regular expression or simple match string
-      required:
-        - matchType
-        - value
     UserIdentifierMatchType:
       description: The type of pattern. For regex, use `EXPRESSION`.
       type: string
@@ -54205,9 +54202,6 @@ components:
             $ref: '#/components/schemas/UserIdentifierConditionEvaluatorPattern'
         type:
           $ref: '#/components/schemas/UserIdentifierType'
-      required:
-        - patterns
-        - type
     UserIdentifierType:
       description: What to match against, either user ID or an attribute in the user's Okta profile.
       type: string

CHANGELOG.md

@@ -1,6 +1,18 @@
 # Changelog
 Running changelog of releases since `2.0.0-rc.4`
 
+## v6.1.6
+- Fix UserIdentifierPolicyRuleCondition by making patterns and type as optional parameters. Thanks [@pranav-okta](https://github.com/pranav-okta)
+
+## v6.1.5
+- Fix the RiskPolicyRuleConditions where level is a required parameter. Thanks [@pranav-okta](https://github.com/pranav-okta)
+
+## v6.1.4
+- Add OAuth2ClientJsonWebKeySet schema for listJwk response. Thanks [@aditya-okta](https://github.com/aditya-okta)
+
+## v6.1.3
+- Fix the CreatePushGroupMapping to accept AppConfig as a pointer. Thanks [@pranav-okta](https://github.com/pranav-okta)
+
 ## v6.1.2
 - Fix AppConfig by adding required type param. Thanks [@pranav-okta](https://github.com/pranav-okta)
 

okta/README.md

@@ -6,7 +6,7 @@ Allows customers to easily access the Okta Management APIs
 This API client was generated by the [OpenAPI Generator](https://openapi-generator.tech) project.  By using the [OpenAPI-spec](https://www.openapis.org/) from a remote server, you can easily generate an API client.
 
 - API version: 5.1.0
-- Package version: 6.1.5
+- Package version: 6.1.6
 - Build package: org.openapitools.codegen.languages.GoClientCodegen
 For more information, please visit [https://developer.okta.com/](https://developer.okta.com/)
 

okta/api/openapi.yaml

@@ -82507,9 +82507,6 @@ components:
         value:
           description: The regular expression or simple match string
           type: string
-      required:
-      - matchType
-      - value
       type: object
     UserIdentifierMatchType:
       description: "The type of pattern. For regex, use `EXPRESSION`."
@@ -82538,9 +82535,6 @@ components:
           x-okta-known-values:
           - ATTRIBUTE
           - IDENTIFIER
-      required:
-      - patterns
-      - type
       type: object
     UserIdentifierType:
       description: "What to match against, either user ID or an attribute in the user's\

okta/client.go

@@ -68,7 +68,7 @@ var (
 )
 
 const (
-	VERSION                   = "6.1.5"
+	VERSION                   = "6.1.6"
 	AccessTokenCacheKey       = "OKTA_ACCESS_TOKEN"
 	DpopAccessTokenNonce      = "DPOP_OKTA_ACCESS_TOKEN_NONCE"
 	DpopAccessTokenPrivateKey = "DPOP_OKTA_ACCESS_TOKEN_PRIVATE_KEY"

okta/configuration.go

@@ -165,7 +165,7 @@ type Configuration struct {
 func NewConfiguration(conf ...ConfigSetter) (*Configuration, error) {
 	cfg := &Configuration{
 		DefaultHeader: make(map[string]string),
-		UserAgent:     fmt.Sprintf("okta-sdk-golang/%s golang/%s %s/%s", "6.1.5", runtime.Version(), runtime.GOOS, runtime.GOARCH),
+		UserAgent:     fmt.Sprintf("okta-sdk-golang/%s golang/%s %s/%s", "6.1.6", runtime.Version(), runtime.GOOS, runtime.GOARCH),
 		Debug:         false,
 		Servers: ServerConfigurations{
 			{

okta/docs/UserIdentifierConditionEvaluatorPattern.md

@@ -4,14 +4,14 @@
 
 Name | Type | Description | Notes
 ------------ | ------------- | ------------- | -------------
-**MatchType** | **string** | The type of pattern. For regex, use `EXPRESSION`. | 
-**Value** | **string** | The regular expression or simple match string | 
+**MatchType** | Pointer to **string** | The type of pattern. For regex, use `EXPRESSION`. | [optional] 
+**Value** | Pointer to **string** | The regular expression or simple match string | [optional] 
 
 ## Methods
 
 ### NewUserIdentifierConditionEvaluatorPattern
 
-`func NewUserIdentifierConditionEvaluatorPattern(matchType string, value string, ) *UserIdentifierConditionEvaluatorPattern`
+`func NewUserIdentifierConditionEvaluatorPattern() *UserIdentifierConditionEvaluatorPattern`
 
 NewUserIdentifierConditionEvaluatorPattern instantiates a new UserIdentifierConditionEvaluatorPattern object
 This constructor will assign default values to properties that have it defined,
@@ -45,6 +45,11 @@ and a boolean to check if the value has been set.
 
 SetMatchType sets MatchType field to given value.
 
+### HasMatchType
+
+`func (o *UserIdentifierConditionEvaluatorPattern) HasMatchType() bool`
+
+HasMatchType returns a boolean if a field has been set.
 
 ### GetValue
 
@@ -65,6 +70,11 @@ and a boolean to check if the value has been set.
 
 SetValue sets Value field to given value.
 
+### HasValue
+
+`func (o *UserIdentifierConditionEvaluatorPattern) HasValue() bool`
+
+HasValue returns a boolean if a field has been set.
 
 
 [[Back to Model list]](../README.md#documentation-for-models) [[Back to API list]](../README.md#documentation-for-api-endpoints) [[Back to README]](../README.md)

okta/docs/UserIdentifierPolicyRuleCondition.md

@@ -5,14 +5,14 @@
 Name | Type | Description | Notes
 ------------ | ------------- | ------------- | -------------
 **Attribute** | Pointer to **string** | The name of the profile attribute to match against. Only used when type is `ATTRIBUTE`. | [optional] 
-**Patterns** | [**[]UserIdentifierConditionEvaluatorPattern**](UserIdentifierConditionEvaluatorPattern.md) |  | 
-**Type** | **string** | What to match against, either user ID or an attribute in the user's Okta profile. | 
+**Patterns** | Pointer to [**[]UserIdentifierConditionEvaluatorPattern**](UserIdentifierConditionEvaluatorPattern.md) |  | [optional] 
+**Type** | Pointer to **string** | What to match against, either user ID or an attribute in the user's Okta profile. | [optional] 
 
 ## Methods
 
 ### NewUserIdentifierPolicyRuleCondition
 
-`func NewUserIdentifierPolicyRuleCondition(patterns []UserIdentifierConditionEvaluatorPattern, type_ string, ) *UserIdentifierPolicyRuleCondition`
+`func NewUserIdentifierPolicyRuleCondition() *UserIdentifierPolicyRuleCondition`
 
 NewUserIdentifierPolicyRuleCondition instantiates a new UserIdentifierPolicyRuleCondition object
 This constructor will assign default values to properties that have it defined,
@@ -71,6 +71,11 @@ and a boolean to check if the value has been set.
 
 SetPatterns sets Patterns field to given value.
 
+### HasPatterns
+
+`func (o *UserIdentifierPolicyRuleCondition) HasPatterns() bool`
+
+HasPatterns returns a boolean if a field has been set.
 
 ### GetType
 
@@ -91,6 +96,11 @@ and a boolean to check if the value has been set.
 
 SetType sets Type field to given value.
 
+### HasType
+
+`func (o *UserIdentifierPolicyRuleCondition) HasType() bool`
+
+HasType returns a boolean if a field has been set.
 
 
 [[Back to Model list]](../README.md#documentation-for-models) [[Back to API list]](../README.md#documentation-for-api-endpoints) [[Back to README]](../README.md)

okta/model_user_identifier_condition_evaluator_pattern.go

@@ -25,7 +25,6 @@ package okta
 
 import (
 	"encoding/json"
-	"fmt"
 )
 
 // checks if the UserIdentifierConditionEvaluatorPattern type satisfies the MappedNullable interface at compile time
@@ -34,9 +33,9 @@ var _ MappedNullable = &UserIdentifierConditionEvaluatorPattern{}
 // UserIdentifierConditionEvaluatorPattern Specifies the details of the patterns to match against
 type UserIdentifierConditionEvaluatorPattern struct {
 	// The type of pattern. For regex, use `EXPRESSION`.
-	MatchType string `json:"matchType"`
+	MatchType *string `json:"matchType,omitempty"`
 	// The regular expression or simple match string
-	Value                string `json:"value"`
+	Value                *string `json:"value,omitempty"`
 	AdditionalProperties map[string]interface{}
 }
 
@@ -46,10 +45,8 @@ type _UserIdentifierConditionEvaluatorPattern UserIdentifierConditionEvaluatorPa
 // This constructor will assign default values to properties that have it defined,
 // and makes sure properties required by API are set, but the set of arguments
 // will change when the set of required properties is changed
-func NewUserIdentifierConditionEvaluatorPattern(matchType string, value string) *UserIdentifierConditionEvaluatorPattern {
+func NewUserIdentifierConditionEvaluatorPattern() *UserIdentifierConditionEvaluatorPattern {
 	this := UserIdentifierConditionEvaluatorPattern{}
-	this.MatchType = matchType
-	this.Value = value
 	return &this
 }
 
@@ -61,52 +58,68 @@ func NewUserIdentifierConditionEvaluatorPatternWithDefaults() *UserIdentifierCon
 	return &this
 }
 
-// GetMatchType returns the MatchType field value
+// GetMatchType returns the MatchType field value if set, zero value otherwise.
 func (o *UserIdentifierConditionEvaluatorPattern) GetMatchType() string {
-	if o == nil {
+	if o == nil || IsNil(o.MatchType) {
 		var ret string
 		return ret
 	}
-
-	return o.MatchType
+	return *o.MatchType
 }
 
-// GetMatchTypeOk returns a tuple with the MatchType field value
+// GetMatchTypeOk returns a tuple with the MatchType field value if set, nil otherwise
 // and a boolean to check if the value has been set.
 func (o *UserIdentifierConditionEvaluatorPattern) GetMatchTypeOk() (*string, bool) {
-	if o == nil {
+	if o == nil || IsNil(o.MatchType) {
 		return nil, false
 	}
-	return &o.MatchType, true
+	return o.MatchType, true
+}
+
+// HasMatchType returns a boolean if a field has been set.
+func (o *UserIdentifierConditionEvaluatorPattern) HasMatchType() bool {
+	if o != nil && !IsNil(o.MatchType) {
+		return true
+	}
+
+	return false
 }
 
-// SetMatchType sets field value
+// SetMatchType gets a reference to the given string and assigns it to the MatchType field.
 func (o *UserIdentifierConditionEvaluatorPattern) SetMatchType(v string) {
-	o.MatchType = v
+	o.MatchType = &v
 }
 
-// GetValue returns the Value field value
+// GetValue returns the Value field value if set, zero value otherwise.
 func (o *UserIdentifierConditionEvaluatorPattern) GetValue() string {
-	if o == nil {
+	if o == nil || IsNil(o.Value) {
 		var ret string
 		return ret
 	}
-
-	return o.Value
+	return *o.Value
 }
 
-// GetValueOk returns a tuple with the Value field value
+// GetValueOk returns a tuple with the Value field value if set, nil otherwise
 // and a boolean to check if the value has been set.
 func (o *UserIdentifierConditionEvaluatorPattern) GetValueOk() (*string, bool) {
-	if o == nil {
+	if o == nil || IsNil(o.Value) {
 		return nil, false
 	}
-	return &o.Value, true
+	return o.Value, true
 }
 
-// SetValue sets field value
+// HasValue returns a boolean if a field has been set.
+func (o *UserIdentifierConditionEvaluatorPattern) HasValue() bool {
+	if o != nil && !IsNil(o.Value) {
+		return true
+	}
+
+	return false
+}
+
+// SetValue gets a reference to the given string and assigns it to the Value field.
 func (o *UserIdentifierConditionEvaluatorPattern) SetValue(v string) {
-	o.Value = v
+	o.Value = &v
 }
 
 func (o UserIdentifierConditionEvaluatorPattern) MarshalJSON() ([]byte, error) {
@@ -119,8 +132,12 @@ func (o UserIdentifierConditionEvaluatorPattern) MarshalJSON() ([]byte, error) {
 
 func (o UserIdentifierConditionEvaluatorPattern) ToMap() (map[string]interface{}, error) {
 	toSerialize := map[string]interface{}{}
-	toSerialize["matchType"] = o.MatchType
-	toSerialize["value"] = o.Value
+	if !IsNil(o.MatchType) {
+		toSerialize["matchType"] = o.MatchType
+	}
+	if !IsNil(o.Value) {
+		toSerialize["value"] = o.Value
+	}
 
 	for key, value := range o.AdditionalProperties {
 		toSerialize[key] = value
@@ -130,28 +147,6 @@ func (o UserIdentifierConditionEvaluatorPattern) ToMap() (map[string]interface{}
 }
 
 func (o *UserIdentifierConditionEvaluatorPattern) UnmarshalJSON(data []byte) (err error) {
-	// This validates that all required properties are included in the JSON object
-	// by unmarshalling the object into a generic map with string keys and checking
-	// that every required field exists as a key in the generic map.
-	requiredProperties := []string{
-		"matchType",
-		"value",
-	}
-
-	allProperties := make(map[string]interface{})
-
-	err = json.Unmarshal(data, &allProperties)
-
-	if err != nil {
-		return err
-	}
-
-	for _, requiredProperty := range requiredProperties {
-		if _, exists := allProperties[requiredProperty]; !exists {
-			return fmt.Errorf("no value given for required property %v", requiredProperty)
-		}
-	}
-
 	varUserIdentifierConditionEvaluatorPattern := _UserIdentifierConditionEvaluatorPattern{}
 
 	err = json.Unmarshal(data, &varUserIdentifierConditionEvaluatorPattern)