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

NSFS | NC | Health | new_buckets_path check access bug fixes #7782

Closed
romayalon opened this issue Feb 1, 2024 · 1 comment
Closed

NSFS | NC | Health | new_buckets_path check access bug fixes #7782

romayalon opened this issue Feb 1, 2024 · 1 comment
Labels
Non Containerized Non containerized NS-FS Release 5.15.z Needs to be included in the 5.15.z release Request Validation

Comments

@romayalon
Copy link
Contributor

Environment info

  • NooBaa Version: master
  • Platform: NC

Actual behavior

  1. The new_buckets_path access check of the account is based on uid/gid of the account, although account can be based on user (distinguished name).
  2. The checkAcceess()/ stat() functions used by the health script to check permissions of an account to its new_buckets_path check only read permissions.

Expected behavior

  1. Health script should be able to check if an account can access new_buckets_path when having user (distinguished_name) instead of uid/gid.
  2. Health script should check also write permissions of an account to its new_buckets_path.

Steps to reproduce

distinguished name issue -

  1. Create an account while specifying a distinguished name instead of uid and gid
  2. Run the health script and check the result.

write permissions issue -

  1. Create an account with only read permissions to its new_buckets_path.
  2. Run the health script and check the result.

More information - Screenshots / Logs / Other output

@romayalon romayalon added the NS-FS label Feb 1, 2024
@romayalon
Copy link
Contributor Author

this was fixed on #7765, adding request validation label

@nimrod-becker nimrod-becker added the Release 5.15.z Needs to be included in the 5.15.z release label Feb 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Non Containerized Non containerized NS-FS Release 5.15.z Needs to be included in the 5.15.z release Request Validation
Projects
None yet
Development

No branches or pull requests

2 participants