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.

Sign up for GitHub

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

Jump to bottom

Drop PUT access from DELETE-related tests #736

Closed
roman-khimov opened this issue Mar 11, 2024 · 1 comment
Closed

Drop PUT access from DELETE-related tests #736

roman-khimov opened this issue Mar 11, 2024 · 1 comment
Assignees
@evgeniiz321
Labels
enhancement Improving existing functionality I3 Minimal impact S4 Routine U1 Critically important to resolve quickly

Comments

@roman-khimov
Copy link
Member

We have a behavior change in nspcc-dev/neofs-node#2261 (to be included in 0.41.0), so while our old test work (yay!) we need to improve those ones that check for ACL restrictions for DELETE (including session/bearer token tests). Currently they all set PUT access at the same time, but with this change we can finally drop it and just leave DELETE access without PUT, it should be sufficient.
@roman-khimov roman-khimov added enhancement Improving existing functionality U1 Critically important to resolve quickly S4 Routine I3 Minimal impact labels Mar 11, 2024
@roman-khimov roman-khimov mentioned this issue Mar 11, 2024
Open
@roman-khimov
Copy link
Member Author

EACL and bearer. Session tokens don't have this problem.

@evgeniiz321 evgeniiz321 mentioned this issue Mar 20, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@evgeniiz321 evgeniiz321

Labels
enhancement Improving existing functionality I3 Minimal impact S4 Routine U1 Critically important to resolve quickly
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@evgeniiz321 @roman-khimov