[#91] Add support namespaces #96
No reviewers
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
4 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: TrueCloudLab/frostfs-http-gw#96
Loading…
Reference in New Issue
There is no content yet.
Delete Branch ":feature/91-add_support_namespaces"
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?
Close #91
Signed-off-by: Roman Loginov r.loginov@yadro.com
@ -318,0 +332,4 @@
|----------------------|------------|---------------|-----------------------|--------------------------------------------------------------------------------------------------------------------------|
| `namespace_header` | `string` | yes | `X-Frostfs-Namespace` | Header to determine zone to resolve bucket name. |
| `allow` | `[]string` | no | | List of container zones which are available to resolve. Mutual exclusive with `deny` list. Prioritized over `deny` list. |
| `default_namespaces` | `[]string` | n/d | ["","root"] | Namespaces that should be handled as default. |
Actually we support sighup reload for
default_namespaces
@ -88,2 +89,4 @@
ctx := utils.GetContextFromRequest(c)
nsBytes := c.Request.Header.Peek(h.config.NamespaceHeader())
ctx = middleware.SetNamespace(ctx, string(nsBytes))
Can we set namespace to context once in some middleware?
@ -184,0 +214,4 @@
}
zone, _ := settings.FormContainerZone(namespace)
d.SetZone(zone)
Can we add some integration test to check this?
For example:
239cbb696d
tocbaa5efc5e
LGTM