Fix gitlab intergation #125
No reviewers
TrueCloudLab/storage-services-developers
Labels
No Label
P0
P1
P2
P3
good first issue
Infrastructure
blocked
bug
config
discussion
documentation
duplicate
enhancement
go
help wanted
internal
invalid
kludge
observability
perfomance
question
refactoring
wontfix
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: TrueCloudLab/frostfs-s3-gw#125
Loading…
Reference in New Issue
There is no content yet.
Delete Branch "dkirillov/frostfs-s3-gw:bugfix/content-length-presigned-url"
Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?
Fix trailing whitespaces
Handle negative Content-Length on put
Add computing actual object size during calculating hash on put.
Use this actual value to save in tree and cache
api/auth: DisableURIPathEscaping for presign
Don't use escaping when presign url.
Escape manually before.
65cb6d7e07
to1381020300
WIP: Fix gitlag intergationto WIP: Fix gitlab intergationWIP: Fix gitlab intergationto Fix gitlab intergation@ -291,6 +291,7 @@ func cloneRequest(r *http.Request, authHeader *authHeader) *http.Request {
func (c *center) checkSign(authHeader *authHeader, box *accessbox.Box, request *http.Request, signatureDateTime time.Time) error {
awsCreds := credentials.NewStaticCredentials(authHeader.AccessKeyID, box.Gate.AccessKey, "")
signer := v4.NewSigner(awsCreds)
signer.DisableURIPathEscaping = true
Is it compatible with
aws s3 presign
?Yes,
aws presign
generates a url like one that we couldn't handle in gitlab integration@ -229,2 +229,4 @@
return nil, err
}
if p.Info.Encryption.Enabled() {
size = decSize
Can you provide some examples why
size
depends onp.Info.Encryption.Enabled()
?Can't we just always use
size
from the line above?Encrypted payload can be different from plain one. So we want to know the real size in HEAD requests for example, so we store real size in object attributes and tree node attribute
Makes sense, thanks.