-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
HistoryController add @Secured #5885
HistoryController add @Secured #5885
Conversation
merge from alibaba:develop
43000ba
to
b36c24c
Compare
@@ -74,7 +78,11 @@ | |||
* @return history config info | |||
*/ | |||
@GetMapping | |||
public ConfigHistoryInfo getConfigHistoryInfo(@RequestParam("nid") Long nid) { | |||
@Secured(action = ActionTypes.READ, parser = ConfigResourceParser.class) | |||
public ConfigHistoryInfo getConfigHistoryInfo(@RequestParam("dataId") String dataId, // for @Secured |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please do not change the paramters
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
persistService find detailConfigHistory add params dataId, group, tenant
@@ -86,7 +94,11 @@ public ConfigHistoryInfo getConfigHistoryInfo(@RequestParam("nid") Long nid) { | |||
* @since 1.4.0 | |||
*/ | |||
@GetMapping(value = "/previous") | |||
public ConfigHistoryInfo getPreviousConfigHistoryInfo(@RequestParam("id") Long id) { | |||
@Secured(action = ActionTypes.READ, parser = ConfigResourceParser.class) | |||
public ConfigHistoryInfo getPreviousConfigHistoryInfo(@RequestParam("dataId") String dataId, // for @Secured |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please do not change the paramters
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
persistService find detailConfigHistory add params dataId, group, tenant
…s dataId, group, tenant
关闭 PR,期望社区商议更优实现 |
feat #5884