Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OKTA-709693: make userschemaattribute minlength maxlength nullable #713

Conversation

bryanapellanes-okta
Copy link
Contributor

@bryanapellanes-okta bryanapellanes-okta commented May 6, 2024

  • Make UserSchemaAttribute.MinLength nullable
  • Make UserSchemaAttribute.MaxLength nullable

Fixes #702

@rcollette
Copy link

@bryanapellanes-okta - I'm not seeing code that would resolve #712 ?

@bryanapellanes-okta
Copy link
Contributor Author

@bryanapellanes-okta - I'm not seeing code that would resolve #712 ?

@rcollette Thanks for your patience on this. I apologize for the oversight, I am reviewing this further to address #712.

@bryanapellanes-okta
Copy link
Contributor Author

@rcollette Upon closer inspection the solution to #712 requires additional consideration. I've removed the "fixes" declaration for #712 from this PR. I will investigate further to determine an appropriate fix for #712.

@laura-rodriguez
Copy link
Collaborator

LGTM, @bryanapellanes-okta please confirm the spec changes with the API team before merging.

@bryanapellanes-okta bryanapellanes-okta merged commit be1cd24 into lr-8.0.0-rc May 16, 2024
1 check passed
@bryanapellanes-okta
Copy link
Contributor Author

LGTM, @bryanapellanes-okta please confirm the spec changes with the API team before merging.

related spec change https://github.com/atko-eng/okta-oas3/pull/983

@bryanapellanes-okta bryanapellanes-okta deleted the OKTA-709693-userschemaattributes-minmaxlength-nullable branch May 16, 2024 18:24
laura-rodriguez added a commit that referenced this pull request May 24, 2024
* Fix "Unable to access IdP links"  (#708)
* Add support for DPoP (#711)
- Add a DPoPProofJwtGenerator class responsbile for DPoP Proof JWT generation
- Update OAuthTokenProvider interface and default classes to return an OAuthTokenResponse instead of just the access token
- Extend the OAuthTokenResponse with OAuthApiResponse properties. This way, we can parse OAuth errors that come with a different format in comparison with Okta resource APIs
- Update the OktaPagedCollectionEnumerator so the DPoP header is added in paginated results when DPoP is enabled
- Update the ApiClient.ExecuteWithRetryHeaders to generate a new DPoP JWT when it's applicable
- Update templates with the proposed DPoP solution and regenerate the SDK to make sure everything is in place
- Add integration tests to cover DPoP scenarios
- Add unit tests to cover DPoP scenarios
- Update readme with a note about DPoP support
* OKTA-709693: make userschemaattribute minlength maxlength nullable (#713)
* Rollback policy changes (#717)
---------

Co-authored-by: Bryan Apellanes <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants