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

composition: Authenticate pallet requirements #243

Open
ethanjli opened this issue Jun 13, 2024 · 0 comments
Open

composition: Authenticate pallet requirements #243

ethanjli opened this issue Jun 13, 2024 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@ethanjli
Copy link
Member

ethanjli commented Jun 13, 2024

Currently, pallet requirements are specified with a version lock which includes a timestamp and commit hash. However, the contents of the locked commit aren't authenticated. The Go Modules system does this with checksums in go.sum files, and we should also compute (and check) checksums when changing the version lock file. Maybe we should ignore any .git directories inside? (What does Go Modules do with .git directories?)

We also need checksums on file downloads. Those checksums should be stored in separate files together with forklift-package.yml files, maybe in a subdirectory of the forklift package, and a subcommand should be provided to update the checksums for a package.

It would be even more secure if we had an equivalent of Go's global checksum database to precompute checksums when a commit is published, but that's a lot of complexity for implementation and infrastructure operations - which outweighs any security concerns we have for now, so that should be considered out-of-scope for this issue.

@ethanjli ethanjli added the enhancement New feature or request label Jun 13, 2024
@ethanjli ethanjli self-assigned this Jun 14, 2024
@ethanjli ethanjli added this to the Preprint Nice-to-Haves milestone Jun 14, 2024
@ethanjli ethanjli moved this to 📋 Plan needed in PlanktoScope Software Jun 14, 2024
@ethanjli ethanjli changed the title Authenticate pallet requirements reqs: Authenticate pallet requirements Jun 14, 2024
@ethanjli ethanjli changed the title reqs: Authenticate pallet requirements composition: Authenticate pallet requirements Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: 📋 Plan needed
Development

No branches or pull requests

1 participant