Expand xmlns field ignore #221
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#221
Loading…
Reference in New Issue
There is no content yet.
Delete Branch "%!s(<nil>)"
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?
Is your feature request related to a problem? Please describe.
In #40 S3 Gateway introduced new
kludge.use_default_xmlns_for_complete_multipart
flag to ignore xmlns field in multipart. Turns out, there are plenty more requests without xmlns field in libraries like minio-go and minio-js:Describe the solution you'd like
Add more generic
kludge
flag to enable and disable all these xmlns checks.I would suggest extending this new flag to all requests rather than just for a little subset. This makes the code more consistent and it seems there is not downside of such solution. Otherwise, there is a possibility that we will encounter a similar problem for other queries in the future.