Skip to content

Conversation

@lvshaokang
Copy link
Contributor

What changes were proposed in this pull request

adjust swagger api display range for security, only retain application op api.

Brief change log

Verifying this change

Does this pull request potentially affect one of the following parts

  • Dependencies (does it add or upgrade a dependency): (yes / no) no

@lvshaokang
Copy link
Contributor Author

cc @wolfboys PTAL.

@caicancai caicancai requested a review from wolfboys April 16, 2024 01:23
Copy link
Member

@caicancai caicancai left a comment

Choose a reason for hiding this comment

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

LGTM

red added 3 commits April 16, 2024 13:50
# Conflicts:
#	streampark-console/streampark-console-service/src/main/java/org/apache/streampark/console/core/controller/SavePointController.java
# Conflicts:
#	streampark-console/streampark-console-service/src/main/java/org/apache/streampark/console/system/controller/AccessTokenController.java
#	streampark-console/streampark-console-service/src/main/java/org/apache/streampark/console/system/controller/UserController.java
@lvshaokang
Copy link
Contributor Author

@wolfboys PTAL. Remove the @Tag/@operation annotations and retain start/cancel as openapi

Copy link
Member

@wolfboys wolfboys left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks for the great work

@wolfboys wolfboys merged commit 1a2af69 into apache:dev-2.1.4 Apr 16, 2024
@lvshaokang lvshaokang deleted the improve/api-scope branch April 22, 2024 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants