Issues with unmarshaling and schema matches in v6

[tim-koehler]

Describe the bug?

This is a followup to #549 regarding issues in the v6 versions of the SDK.

Examples:

clientV6.ApplicationAPI.ListApplications(ctx).Execute()

failed to unmarshal ListApplications200ResponseInner as SamlApplication: no value given for required property allowMultipleAcsEndpoints"

or

clientV6.IdentityProviderAPI.ListIdentityProviders(ctx).Execute()

data matches more than one schema in oneOf(IdentityProviderProtocol)

What is expected to happen?

Proper unmarshaling without errors.

What is the actual behavior?

Errors like shown in the description

Reproduction Steps?

Invoking the above mentioned SDK functions with at least one SAML application or an existing SAML IDP

Additional Information?

No response

Golang Version

v1.25.8

SDK Version

v6.0.1

OS version

No response

[tim-koehler]

@pranav-okta New ticket created.

Thanks

[pranav-okta]

@tim-koehler The Identity Provider API bug is fixed now.

The other bug is due to a change in the fields from optional to required. There is an open PR, but I need to verify the APIs to see whether these changes are intentional.

[aditya-okta]

@tim-koehler the SAML application bug has been fixed in the latest release v6.0.3. Please verify once.

[tim-koehler]

@aditya-okta
I temporarily don't have access to the the Okta tenant where I can verify that this issue is fixed. However, I can answer that question in the next one or two weeks.

[srikkanthm]

@aditya-okta I still see the issue in v6.0.3

[aditya-okta]

@srikkanthm , any particular API for which you are seeing this issue?

[erezrokah]

Hi @aditya-okta this PR #559 shows a bunch of fields in various models that are marked as required in v6 but shouldn't be (they are missing from API responses).

See also #559 (comment) with the exact same error as in this issue

[srikkanthm]

@aditya-okta it's ListApplications API. I can provide more details later.

[srikkanthm]

@aditya-okta I think it's the org2org application that's failing to unmarshall. I see the fix didn't include all the application types.

[srikkanthm]

@aditya-okta I'm still seeing these issues even with the latest version from last week.