server: allow non-admin maintenance status

kunal.behbudzade · Deln0r · commit 576a6a00cdde · 2026-05-26T17:46:32.000+08:00
Signed-off-by: kunal.behbudzade &lt;kunal.behbudzade@btsgrp.com&gt;
Signed-off-by: Ian Chechin &lt;ian00chechin@gmail.com&gt;
diff --git a/server/etcdserver/api/v3rpc/maintenance.go b/server/etcdserver/api/v3rpc/maintenance.go
@@ -364,7 +364,7 @@ func (ams *authMaintenanceServer) Alarm(ctx context.Context, ar *pb.AlarmRequest
 }
 
 func (ams *authMaintenanceServer) Status(ctx context.Context, ar *pb.StatusRequest) (*pb.StatusResponse, error) {
-	if err := ams.isPermitted(ctx); err != nil {
+	if err := ams.requireAuthInfo(ctx); err != nil {
 		return nil, togRPCError(err)
 	}
 
diff --git a/tests/common/maintenance_auth_test.go b/tests/common/maintenance_auth_test.go
@@ -166,7 +166,7 @@ func TestStatusWithRootAuth(t *testing.T) {
 }
 
 func TestStatusWithUserAuth(t *testing.T) {
-	testStatusWithAuth(t, false, true, WithAuth("user0", "user0Pass"))
+	testStatusWithAuth(t, false, false, WithAuth("user0", "user0Pass"))
 }
 
 func testStatusWithAuth(t *testing.T, expectConnectionError, expectOperationError bool, opts ...config.ClientOption) {

Original file line number	Diff line number	Diff line change
`@@ -364,7 +364,7 @@ func (ams authMaintenanceServer) Alarm(ctx context.Context, ar pb.AlarmRequest`
`364`	`364`	`}`
`365`	`365`
`366`	`366`	`func (ams authMaintenanceServer) Status(ctx context.Context, ar pb.StatusRequest) (*pb.StatusResponse, error) {`
`367`		`- if err := ams.isPermitted(ctx); err != nil {`
	`367`	`+ if err := ams.requireAuthInfo(ctx); err != nil {`
`368`	`368`	`return nil, togRPCError(err)`
`369`	`369`	`}`
`370`	`370`
Original file line number	Diff line number	Diff line change
`@@ -166,7 +166,7 @@ func TestStatusWithRootAuth(t *testing.T) {`
`166`	`166`	`}`
`167`	`167`
`168`	`168`	`func TestStatusWithUserAuth(t *testing.T) {`
`169`		`- testStatusWithAuth(t, false, true, WithAuth("user0", "user0Pass"))`
	`169`	`+ testStatusWithAuth(t, false, false, WithAuth("user0", "user0Pass"))`
`170`	`170`	`}`
`171`	`171`
`172`	`172`	`func testStatusWithAuth(t *testing.T, expectConnectionError, expectOperationError bool, opts ...config.ClientOption) {`