Skip to content

[Experiment] Add OpenAPI request response validation#1228

Closed
arvindkrishnakumar-okta wants to merge 5 commits intomasterfrom
ak_add_openapi_req_resp_validation
Closed

[Experiment] Add OpenAPI request response validation#1228
arvindkrishnakumar-okta wants to merge 5 commits intomasterfrom
ak_add_openapi_req_resp_validation

Conversation

@arvindkrishnakumar-okta
Copy link
Copy Markdown
Contributor

@arvindkrishnakumar-okta arvindkrishnakumar-okta commented May 10, 2024

Issue(s)

OKTA-727240

Description

Add Open API request/response validation of the spec against live Okta backend.

Category

  • Bugfix
  • Enhancement
  • New Feature
  • Library Upgrade
  • Configuration Change
  • Versioning Change
  • Unit or Integration Test(s)
  • Documentation

Signoff

  • I have submitted a CLA for this PR
  • Each commit message explains what the commit does
  • I have updated documentation to explain what my PR does
  • My code is covered by tests if required
  • I did not edit any automatically generated files

Comment thread src/swagger/api.yaml
ids:
- 00g4eralvekR5RLuS0g7
- 00g4eralvekR5RLuS0g8
# policyContext:
Copy link
Copy Markdown
Contributor Author

@arvindkrishnakumar-okta arvindkrishnakumar-okta May 10, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a spec change that needs to be fixed (it does not work well with Java SDK presently). This has emerged now since we're validating the spec now.

@monde
Copy link
Copy Markdown

monde commented May 10, 2024

❤️

@arvindkrishnakumar-okta arvindkrishnakumar-okta marked this pull request as ready for review May 17, 2024 14:31
@arvindkrishnakumar-okta arvindkrishnakumar-okta changed the title [DRAFT] Add OpenAPI request response validation [Experiment] Add OpenAPI request response validation Sep 3, 2024
@prachi-okta prachi-okta closed this Mar 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants