Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Should a node validate objects it won't store? #2793

Open
carpawell opened this issue Mar 29, 2024 · 1 comment
Open

Should a node validate objects it won't store? #2793

carpawell opened this issue Mar 29, 2024 · 1 comment
Labels
discussion Open discussion of some problem I4 No visible changes S1 Highly significant U3 Regular

Comments

@carpawell
Copy link
Member

Now any SN tries to validate and check eACL rules on every request (even if it does not store/will not store an object). Should it do it? Sometimes it cannot even validate smth it does not have access to: #2792.

Possible Solution

If a node is just a middleware in the chain b/w a client and a server, just do what this node should do (split objects, forward requests, etc) and do not try to act like a target node.

Context

Object content validation is here, while placement creation logic (that can say where to put (mb this node is not part of a container at all)) is in the iteratePlacement function.

Your Environment

5c10c503ce7a9be8f25d53972ace416541c80f75

@carpawell carpawell added question Further information is requested discussion Open discussion of some problem labels Mar 29, 2024
@roman-khimov roman-khimov added U3 Regular S1 Highly significant I4 No visible changes and removed question Further information is requested labels Apr 1, 2024
@roman-khimov
Copy link
Member

Non-container nodes better just forward requests and not mess with them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion Open discussion of some problem I4 No visible changes S1 Highly significant U3 Regular
Projects
None yet
Development

No branches or pull requests

2 participants