8000 Add OpenAPI annotation to JSON object of ClientPolicycy representations by hrttthr · Pull Request #39240 · keycloak/keycloak · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add OpenAPI annotation to JSON object of ClientPolicycy representations #39240

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

Merged
merged 1 commit into from
Apr 29, 2025

Conversation

hrttthr
Copy link
Contributor
@hrttthr hrttthr commented Apr 28, 2025

The OpenAPI specification error of missing items, which occurred because the variable configuration was an array type, has been resolved by making configuration an object type.

Closes #32600

Closes keycloak#32600

Signed-off-by: Tetsuhiro Hiruta <tetsuhiro.hiruta.sg@hitachi.com>
@hrttthr hrttthr requested a review from a team as a code owner April 28, 2025 03:35
@tnorimat
Copy link
Contributor

@hrttthr Hello, thank you for the PR. It seems that the PR lacks its integration tests. Could you tell me the reaon why? or simply miss to implement the test?

@hrttthr
Copy link
Contributor Author
hrttthr commented Apr 28, 2025

@tnorimat Thank you for your review. This PR only adds annotations to modify the OpenAPI specification. Therefore, I think it does not affect the operation and does not require testing.
As an alternative, I tested it with swagger as mentioned in the issue #32600 and confirmed that the relevant error did not occur.

@mposolda mposolda self-assigned this Apr 28, 2025
Copy link
Contributor
@tnorimat tnorimat left a comment

Choose a reason for hiding this comment

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

LGTM

@tnorimat
Copy link
Contributor

@hrttthr Thank you for the clarification.
@abstractj @mposolda If possible, could you check the PR?

Copy link
Contributor
@mposolda mposolda left a comment

Choose a reason for hiding this comment

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

@hrttthr @tnorimat Thanks for the updates and review!

@mposolda mposolda merged commit 10c0c8f into keycloak:main Apr 29, 2025
76 checks passed
@tnorimat tnorimat deleted the openapi_fix_missing_items branch May 21, 2025 05:11
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.

OpenAPI spec: Missing attributes in ClientPolicyConditionRepresentation and ClientPolicyExecutorRepresentation schemas
3 participants
0