Create container with EACL #131

New issue

Open

opened 2025-12-28 18:12:40 +00:00 by sami · 2 comments

sami commented

2025-12-28 18:12:40 +00:00

Owner

Originally created by @roman-khimov on GitHub (Aug 6, 2025).

I'm always frustrated when I think of create+setEACL pair. It's error-prone and racy in many ways. There is always a window of time when container is completely open between creation and EACL setting.

Describe the solution you'd like

Provide function to do both atomically. Unfortunately it also requires NeoFS API changes.

Describe alternatives you've considered

Leave this source of trouble.

Additional context

https://github.com/nspcc-dev/neofs-s3-gw/issues/1184

Originally created by @roman-khimov on GitHub (Aug 6, 2025). ## Is your feature request related to a problem? Please describe. I'm always frustrated when I think of create+setEACL pair. It's error-prone and racy in many ways. There is _always_ a window of time when container is completely open between creation and EACL setting. ## Describe the solution you'd like Provide function to do both atomically. Unfortunately it also requires NeoFS API changes. ## Describe alternatives you've considered Leave this source of trouble. ## Additional context https://github.com/nspcc-dev/neofs-s3-gw/issues/1184

sami added the

feature

labels

2025-12-28 18:12:40 +00:00

sami commented

2025-12-28 18:12:40 +00:00

Author

Owner

@roman-khimov commented on GitHub (Dec 5, 2025):

We can do it at the FS API level as well, a transaction can have multiple calls without any problem, so contract interface can be left as is while nodes will create a transaction that does both things.

@roman-khimov commented on GitHub (Dec 5, 2025): We can do it at the FS API level as well, a transaction can have multiple calls without any problem, so contract interface can be left as is while nodes will create a transaction that does both things.

sami commented

2025-12-28 18:12:40 +00:00

Author

Owner

@roman-khimov commented on GitHub (Dec 8, 2025):

Extend creation API to accept EACL as well.

@roman-khimov commented on GitHub (Dec 8, 2025): Extend creation API to accept EACL as well.