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

Status code for ACL access restrcitction #189

Closed
alexvanin opened this issue Jan 21, 2022 · 2 comments · Fixed by #192
Closed

Status code for ACL access restrcitction #189

alexvanin opened this issue Jan 21, 2022 · 2 comments · Fixed by #192
Assignees
@fyrchik

Comments

@alexvanin
Copy link
Contributor

I guess we can have one or two (for basic and extended ACL) status codes that notify about access restriction. It can be used in nspcc-dev/neofs-s3-gw#328 to return correct status code (now it relies on message with access to operation * is denied by sub string).
@KirillovDenis
Copy link

It would be nice to have status code for not found container and object errors as well.
I'm not sure if we need separate issue for this.

@alexvanin
Copy link
Contributor Author

It would be nice to have status code for not found container and object errors as well. I'm not sure if we need separate issue for this.

Better to create new issues in neofs-api repository for that.

@KirillovDenis KirillovDenis mentioned this issue Jan 27, 2022
Closed
@alexvanin alexvanin mentioned this issue Feb 7, 2022
Closed
@fyrchik fyrchik mentioned this issue Feb 9, 2022
Merged
cthulhu-rider pushed a commit that referenced this issue Feb 21, 2022
@fyrchik @cthulhu-rider
[#189] status: add Object section and ACL_DENIED code
5f3fa3f 
Signed-off-by: Evgenii Stratonikov <evgeniy@nspcc.ru>
cthulhu-rider pushed a commit to cthulhu-rider/neofs-api that referenced this issue Feb 21, 2022
[nspcc-dev#189] status: Refine format of the ACCESS_DENIED status d…
3ee8fd9 
…etail

Signed-off-by: Leonard Lyubich <leonard@nspcc.ru>
cthulhu-rider pushed a commit that referenced this issue Feb 22, 2022
@cthulhu-rider
[#189] status: Refine format of the ACCESS_DENIED status detail
83b0eef 
Signed-off-by: Leonard Lyubich <leonard@nspcc.ru>
@cthulhu-rider cthulhu-rider added this to the v2.12.0 milestone Feb 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fyrchik fyrchik

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add status codes for object, container and session services. fyrchik/frostfs-api
4 participants
@alexvanin @KirillovDenis @fyrchik @cthulhu-rider