Merge pull request #288 from versity/fix/sel-obj-cont-header

SelectObjectContent Content-Length header bug
This commit is contained in:
Ben McClelland
2023-10-13 10:43:34 -07:00
committed by GitHub
2 changed files with 4 additions and 3 deletions

View File

@@ -889,7 +889,7 @@ func (c S3ApiController) CreateActions(ctx *fiber.Ctx) error {
})
}
if err := auth.VerifyACL(parsedAcl, acct.Access, "WRITE", isRoot); err != nil {
if err := auth.VerifyACL(parsedAcl, acct.Access, "READ", isRoot); err != nil {
return SendXMLResponse(ctx, nil, err, &MetaOpts{Logger: c.logger, Action: "SelectObjectContent", BucketOwner: parsedAcl.Owner})
}
@@ -1042,6 +1042,7 @@ func SendXMLResponse(ctx *fiber.Ctx, resp any, err error, l *MetaOpts) error {
}
if len(b) > 0 {
ctx.Response().Header.Set("Content-Length", fmt.Sprint(len(b)))
ctx.Response().Header.SetContentType(fiber.MIMEApplicationXML)
}
}

View File

@@ -131,8 +131,8 @@ type SelectObjectContentResult struct {
Records *types.RecordsEvent
Stats *types.StatsEvent
Progress *types.ProgressEvent
Cont *string
End *string
Cont *types.ContinuationEvent
End *types.EndEvent
}
type Bucket struct {