-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
fix(snapshot): grant execution permissions to the snapshot dir for the owner #18294
fix(snapshot): grant execution permissions to the snapshot dir for the owner #18294
Conversation
WalkthroughThe recent changes primarily focus on enhancing the security and accessibility of the snapshot feature. The permissions for the snapshot directory have been modified to allow execution by the owner, thereby ensuring better control and functionality. Changes
TipsChat with CodeRabbit Bot (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review Status
Actionable comments generated: 0
Configuration used: CodeRabbit UI
Files selected for processing (2)
- CHANGELOG.md (1 hunks)
- server/util.go (1 hunks)
Files skipped from review due to trivial changes (2)
- CHANGELOG.md
- server/util.go
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CHANGELOG.md
Outdated
@@ -76,6 +76,7 @@ Ref: https://keepachangelog.com/en/1.0.0/ | |||
* (x/gov) [#17873](https://github.com/cosmos/cosmos-sdk/pull/17873) Fail any inactive and active proposals whose messages cannot be decoded. | |||
* (simulation) [#17911](https://github.com/cosmos/cosmos-sdk/pull/17911) Fix all problems with executing command `make test-sim-custom-genesis-fast` for simulation test. | |||
* (simulation) [#18196](https://github.com/cosmos/cosmos-sdk/pull/18196) Fix the problem of `validator set is empty after InitGenesis` in simulation test. | |||
* (snapshot) [#18294](https://github.com/cosmos/cosmos-sdk/pull/18294) Grant execution permissions to the snapshot dir for the owner. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* (snapshot) [#18294](https://github.com/cosmos/cosmos-sdk/pull/18294) Grant execution permissions to the snapshot dir for the owner. |
No changelog needed as this regression never been released (#18206)
…e owner (#18294) Co-authored-by: Julien Robert <[email protected]> (cherry picked from commit a378303)
…e owner (#18294) Co-authored-by: Julien Robert <[email protected]> (cherry picked from commit a378303)
…e owner (#18294) Co-authored-by: Julien Robert <[email protected]>
Description
In PR #18206 , the permissions of snapshot dir were modified. However, in my macos sonoma 14, when executing simd start --home ./nodes/node0/simd the following error occurred: panic: failed to initialize database: stat nodes/node0/simd /data/snapshots/metadata.db: permission denied. The main reason is lack of execute permissions for the owner.
In addition, search for the keyword os.MkdirAll in the project, and the owner will have executable permissions.
Summary by CodeRabbit