From 321a9219f684bf85267c6c64bbbb8fc5385a0447 Mon Sep 17 00:00:00 2001 From: Ishank Arora Date: Tue, 5 Oct 2021 12:05:35 +0200 Subject: [PATCH] Return invalid file ID in appprovider --- internal/http/services/appprovider/appprovider.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/internal/http/services/appprovider/appprovider.go b/internal/http/services/appprovider/appprovider.go index 30f949a714..e4dc55d57d 100644 --- a/internal/http/services/appprovider/appprovider.go +++ b/internal/http/services/appprovider/appprovider.go @@ -332,12 +332,12 @@ func (s *svc) getStatInfo(ctx context.Context, fileID string, client gateway.Gat decodedID, err := base64.URLEncoding.DecodeString(fileID) if err != nil { - return nil, ocmd.APIErrorInvalidParameter, errors.Wrap(err, "fileID doesn't follow the required format") + return nil, ocmd.APIErrorInvalidParameter, errors.Wrap(err, fmt.Sprintf("fileID %s doesn't follow the required format", fileID)) } parts := strings.Split(string(decodedID), idDelimiter) if !utf8.ValidString(parts[0]) || !utf8.ValidString(parts[1]) { - return nil, ocmd.APIErrorInvalidParameter, errors.New("fileID contains illegal characters") + return nil, ocmd.APIErrorInvalidParameter, errtypes.BadRequest(fmt.Sprintf("fileID %s contains illegal characters", fileID)) } res := &provider.ResourceId{ StorageId: parts[0],