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.

Sign up for GitHub

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

Jump to bottom

Disable node-status by default? #2952

Closed
spacewander opened this issue Dec 3, 2020 · 1 comment · Fixed by #2968
Closed

Disable node-status by default? #2952

spacewander opened this issue Dec 3, 2020 · 1 comment · Fixed by #2968

Comments

@spacewander
Copy link
Member

This plugin leaks some metrics about the APISIX instance, and is not protected by default.
@moonming
Copy link
Member

moonming commented Dec 3, 2020

LGTM

spacewander added a commit to spacewander/incubator-apisix that referenced this issue Dec 4, 2020
@spacewander
feat: disable node-status by default
68c5c71 
Fix apache#2952
@spacewander spacewander mentioned this issue Dec 4, 2020
Merged
4 tasks
spacewander added a commit to spacewander/incubator-apisix that referenced this issue Dec 4, 2020
@spacewander
feat: disable node-status by default
fa615fc 
Fix apache#2952
tokers pushed a commit that referenced this issue Dec 4, 2020
@spacewander
feat: disable node-status by default (#2968)
2f3ab6a 
Fix #2952
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: disable node-status by default spacewander/incubator-apisix
2 participants
@spacewander @moonming