From eec359cfa85ead558104f6396d5a6eb1d54685f5 Mon Sep 17 00:00:00 2001 From: Airat Arifullin Date: Wed, 4 Sep 2024 15:57:34 +0300 Subject: [PATCH] [#1351] apemanager: Fix `AddChain` handler for audit middleware * `GetChainID` from `frostfs-api-go/v2` does not handler nil-pointer response body that leads to the panic after its call. Signed-off-by: Airat Arifullin --- pkg/services/apemanager/audit.go | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/pkg/services/apemanager/audit.go b/pkg/services/apemanager/audit.go index d132ae7db..73d9e3690 100644 --- a/pkg/services/apemanager/audit.go +++ b/pkg/services/apemanager/audit.go @@ -33,10 +33,15 @@ func (a *auditService) AddChain(ctx context.Context, req *apemanager.AddChainReq return res, err } + var respChainID []byte + if respBody := res.GetBody(); respBody != nil { + respChainID = respBody.GetChainID() + } + audit.LogRequest(a.log, ape_grpc.APEManagerService_AddChain_FullMethodName, req, audit.TargetFromChainID(req.GetBody().GetTarget().GetTargetType().String(), req.GetBody().GetTarget().GetName(), - res.GetBody().GetChainID()), + respChainID), err == nil) return res, err