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

NC | Backport to stage_5.15.3 #8004

Merged

Conversation

romayalon
Copy link
Contributor

shirady and others added 4 commits May 2, 2024 15:54
…t has rw_access to underlying path

1. Add a check if the owner of the bucket has rw_access to the underlying path when creating or updating a bucket.
2. Move the functionality of verify_bucket_owner to validate_bucket_args before we check the access of the underlying path (so we will have the account's details - to reduce fetching the config file again).
3. Add a new error: InaccessibleStoragePath.
4. Edit the existing tests to permit read and write access to the bucket owner.
5. Minor style changes and renaming in test_nc_nsfs_bucket_cli.test.js.

Signed-off-by: shirady <57721533+shirady@users.noreply.github.com>
(cherry picked from commit b558799)
Signed-off-by: Romy <35330373+romayalon@users.noreply.github.com>
(cherry picked from commit 6ce452b)
Signed-off-by: Romy <35330373+romayalon@users.noreply.github.com>

NC - fix server ip whitelist validation

Signed-off-by: Romy <35330373+romayalon@users.noreply.github.com>
(cherry picked from commit 167ed0d)
Signed-off-by: Romy <35330373+romayalon@users.noreply.github.com>

fix code review

Signed-off-by: Romy <35330373+romayalon@users.noreply.github.com>
(cherry picked from commit f3b303a)
@romayalon romayalon force-pushed the romy-backport-stage_5.15.3-encryption branch from 3184e54 to 05d6c49 Compare May 2, 2024 12:55
Signed-off-by: Romy <35330373+romayalon@users.noreply.github.com>
@romayalon romayalon requested a review from nimrod-becker May 2, 2024 13:31
@romayalon romayalon merged commit 59144fb into noobaa:stage_5.15.3 May 2, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants