Skip to content

Conversation

@ddhodge
Copy link
Contributor

@ddhodge ddhodge commented Dec 16, 2021

New features for release

@ddhodge ddhodge requested a review from polarweasel December 16, 2021 21:42
@ddhodge ddhodge self-assigned this Dec 16, 2021
@ddhodge ddhodge added the area/documentation Documentation needed label Dec 16, 2021
@netlify
Copy link

netlify bot commented Dec 16, 2021

✔️ Deploy Preview for infallible-bardeen-164bc9 ready!

🔨 Explore the source changes: c7e1081

🔍 Inspect the deploy log: https://app.netlify.com/sites/infallible-bardeen-164bc9/deploys/61bbc229ec457e00070ed960

😎 Browse the preview: https://deploy-preview-10901--infallible-bardeen-164bc9.netlify.app

Copy link
Contributor

@polarweasel polarweasel left a comment

Choose a reason for hiding this comment

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

Looks good. Left a couple of comments, but you don't need to get another review after you adjust...

@ddhodge ddhodge merged commit bc8500d into yugabyte:master Dec 16, 2021
@ddhodge ddhodge deleted the docs_cloud_newfeatures branch December 16, 2021 23:14
Arpit-yb added a commit that referenced this pull request Nov 14, 2025
…arta namespace

Summary:
Current openapi-generator version only supports the javax namespace, which is used by Spring v5.x. However, as part of [[ https://yugabyte.atlassian.net/browse/PLAT-18516 | PLAT-18516 ]], we need to upgrade to Spring 6.x since 5.x is already EOLed.

We are updating the openapi-generator templates to version 7.17. This update is being done by inspecting the commits went between 5.x and 7.17 and applying them manually to each file.

Version 7.17 also introduces an in-built way to perform OAuth 0 token validation (ref: PR [[ OpenAPITools/openapi-generator#10901 | #10901) ]]. However, this feature has been commented out for now since YBA already relies on pac4j for authentication.

Test Plan:
Tested locally
dev-itests
unit tests
v2 iTests

Reviewers: #yba-api-review, sneelakantan, dkumar, aaryan.chauhan

Reviewed By: #yba-api-review, sneelakantan

Subscribers: yugaware

Differential Revision: https://phorge.dev.yugabyte.com/D48223
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/documentation Documentation needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants