Skip to content
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

JRaft ops interface should require authority #4917

Closed
haoyann opened this issue Feb 21, 2021 · 0 comments · Fixed by #4919
Closed

JRaft ops interface should require authority #4917

haoyann opened this issue Feb 21, 2021 · 0 comments · Fixed by #4919
Labels
contribution welcome kind/enhancement Category issues or prs related to enhancement.
Milestone

Comments

@haoyann
Copy link
Collaborator

haoyann commented Feb 21, 2021

Is your feature request related to a problem? Please describe.
Anyone can request this interface, permission should be required.

    @PostMapping(value = "/raft")
    public RestResult<String> raftOps(@RequestBody Map<String, String> commands) {
        return protocolManager.getCpProtocol().execute(commands);
    }

Describe the solution you'd like
Only the administrator can request this interface.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

@KomachiSion KomachiSion added kind/enhancement Category issues or prs related to enhancement. contribution welcome labels Feb 22, 2021
@KomachiSion KomachiSion linked a pull request Feb 23, 2021 that will close this issue
5 tasks
@KomachiSion KomachiSion added this to the 1.4.2 milestone Feb 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution welcome kind/enhancement Category issues or prs related to enhancement.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants